[llvm] [BOLT][DWARF] Refactor legacy ranges writers (PR #96006)
Maksim Panchenko via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 25 12:08:32 PDT 2024
maksfb wrote:
> hmm I was thinking, maybe as a followup PR. DIE in DWARFRagnes writer doesn't make sense for the instance that handles main binary section. Also since for Rangelistswriter inherits from it, for DWARF5 there is a super slight size penalty. Maybe worth creating another class that inherits from DWARFRangeswriter. Sticking DIE and access APIs there. Storing that instance in the legacy map. @maksfb WDYT?
Don't think it's worth it for one field in a class without a ridiculously large number of instances.
https://github.com/llvm/llvm-project/pull/96006
More information about the llvm-commits
mailing list