This is an archive of the discontinued LLVM Phabricator instance.

[dwarfdump][dwarf] Fix parsing of tu-index
ClosedPublic

Authored by ayermolo on Apr 17 2023, 4:00 PM.

Details

Summary

Fixed issue where {tu,cu}-index fixup code for DWARF5 that would report an error when
fixup map is empty. Which is the case when seciton(s) are not over 4GB or
--manaully-generate-unit-index is not specified

Diff Detail

Event Timeline

ayermolo created this revision.Apr 17 2023, 4:00 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 17 2023, 4:00 PM
Herald added subscribers: hoy, modimo, wenlei and 2 others. · View Herald Transcript
ayermolo requested review of this revision.Apr 17 2023, 4:00 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 17 2023, 4:00 PM
dblaikie added inline comments.Apr 18 2023, 2:35 PM
llvm/test/DebugInfo/X86/dwp-v5-tu-index.s
9

Perhaps this should be a bit less specific, otherwise it might easily be missed - maybe use an --implicit-check-not in the FileCheck command? (like --implicit-check-not=error)

ayermolo updated this revision to Diff 515142.Apr 19 2023, 5:02 PM

addressed comments

ayermolo marked an inline comment as done.Apr 19 2023, 5:03 PM
dblaikie accepted this revision.Apr 20 2023, 10:52 AM

Looks good, thanks!

This revision is now accepted and ready to land.Apr 20 2023, 10:52 AM
This revision was landed with ongoing or failed builds.Apr 20 2023, 11:42 AM
This revision was automatically updated to reflect the committed changes.