[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