This is an archive of the discontinued LLVM Phabricator instance.

[BOLT][DWARF] Change loclist encoding to use base_addrx
ClosedPublic

Authored by ayermolo on Jan 17 2023, 2:51 PM.

Details

Summary

Doing the same thing as for rangelists. Changing loclists to use base_addrx, it
slightly increases .debug_loclists, but reduces .debug_addr section.

sectionclang-16.bolt.baseclang-16.boltraw%
debug_loclists1982082033985190102%
.debug_addr1441580814351448-6436099.5%

Diff Detail

Event Timeline

ayermolo created this revision.Jan 17 2023, 2:51 PM
Herald added a reviewer: Amir. · View Herald Transcript
Herald added a reviewer: maksfb. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
ayermolo requested review of this revision.Jan 17 2023, 2:51 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 17 2023, 2:51 PM
ayermolo edited the summary of this revision. (Show Details)Jan 17 2023, 2:54 PM
ayermolo edited the summary of this revision. (Show Details)Jan 17 2023, 2:58 PM
maksfb accepted this revision.Jan 18 2023, 7:06 PM

s/[bolt]/[BOLT]/

LGTM with nits.

bolt/lib/Core/DebugData.cpp
215

s/ListEntries/ListEntry/ ?

219
This revision is now accepted and ready to land.Jan 18 2023, 7:06 PM
ayermolo retitled this revision from [bolt][dwarf] Change loclist encoding to use base_addrx to [BOLT][DWARF] Change loclist encoding to use base_addrx.Jan 19 2023, 2:26 PM
ayermolo updated this revision to Diff 490664.Jan 19 2023, 2:31 PM

fixed nits

ayermolo marked 2 inline comments as done.Jan 19 2023, 2:31 PM
This revision was automatically updated to reflect the committed changes.