Page MenuHomePhabricator

[DWARF] Support 'G' in dwarf parser
ClosedPublic

Authored by fmayer on Jun 6 2022, 5:57 PM.

Diff Detail

Unit TestsFailed

TimeTest
60,100 msx64 debian > AddressSanitizer-x86_64-linux.TestCases::scariness_score_test.cpp
Script: -- : 'RUN: at line 4'; /var/lib/buildkite-agent/builds/llvm-project/build/./bin/clang --driver-mode=g++ -fsanitize=address -mno-omit-leaf-frame-pointer -fno-omit-frame-pointer -fno-optimize-sibling-calls -gline-tables-only -m64 -O0 /var/lib/buildkite-agent/builds/llvm-project/compiler-rt/test/asan/TestCases/scariness_score_test.cpp -o /var/lib/buildkite-agent/builds/llvm-project/build/projects/compiler-rt/test/asan/X86_64LinuxConfig/TestCases/Output/scariness_score_test.cpp.tmp
30 msx64 debian > LLVM.DebugInfo/AArch64::stack-tagging-cfi.s
Script: -- : 'RUN: at line 1'; /var/lib/buildkite-agent/builds/llvm-project/build/bin/llvm-mc -filetype=obj -triple aarch64-generic-linux -mattr=+mte /var/lib/buildkite-agent/builds/llvm-project/llvm/test/DebugInfo/AArch64/stack-tagging-cfi.s -o /var/lib/buildkite-agent/builds/llvm-project/build/test/DebugInfo/AArch64/Output/stack-tagging-cfi.s.tmp.o

Event Timeline

fmayer created this revision.Jun 6 2022, 5:57 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 6 2022, 5:57 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
fmayer requested review of this revision.Jun 6 2022, 5:57 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 6 2022, 5:57 PM
MaskRay accepted this revision.Jun 6 2022, 7:30 PM

LGTM.

llvm/test/DebugInfo/AArch64/stack-tagging-cfi.s
1

Use an generic ELF target triple to indicate that this does not need Linux features.

2

llvm-dwarfdump --eh-frame %t.o

19

Remove

This revision is now accepted and ready to land.Jun 6 2022, 7:30 PM
MaskRay added inline comments.Jun 6 2022, 7:30 PM
llvm/lib/DebugInfo/DWARF/DWARFDebugFrame.cpp
1144

Can reuse break attached to case 'B'

fmayer updated this revision to Diff 434925.Jun 7 2022, 1:18 PM

address comments

fmayer added inline comments.Jun 7 2022, 1:20 PM
llvm/lib/DebugInfo/DWARF/DWARFDebugFrame.cpp
1144

Is that actually worth saving one line for potential confusion down the line if anyone were ever to add anything to one of those?

fmayer marked 3 inline comments as done.Jun 7 2022, 1:23 PM
fmayer added a subscriber: eugenis.Jun 7 2022, 1:37 PM
MaskRay accepted this revision.Jun 7 2022, 1:45 PM
MaskRay added inline comments.
llvm/test/DebugInfo/AArch64/stack-tagging-cfi.s
19

Delete comments like // -- End function

fmayer updated this revision to Diff 434942.Jun 7 2022, 1:57 PM

address comment

fmayer marked an inline comment as done.Jun 7 2022, 1:58 PM
This revision was landed with ongoing or failed builds.Jun 8 2022, 12:39 PM
This revision was automatically updated to reflect the committed changes.
fmayer reopened this revision.Jun 8 2022, 1:53 PM

Tests broke on windows.

This revision is now accepted and ready to land.Jun 8 2022, 1:53 PM
fmayer updated this revision to Diff 435326.Jun 8 2022, 2:21 PM

require aarch64-registered-target

This revision was landed with ongoing or failed builds.Jun 8 2022, 2:28 PM
This revision was automatically updated to reflect the committed changes.