mstorsjo added a comment. In D70840#1763705 <https://reviews.llvm.org/D70840#1763705>, @mstorsjo wrote:
> In D70840#1763639 <https://reviews.llvm.org/D70840#1763639>, @labath wrote: > > > Yeah, this is going to be tricky... I don't really know what's the right > > way to do this, but here are the thoughts I have on this so far: > > > > - The new DWARFDebugInfoEntry member is going to substantially increase our > > memory footprint -- that's a non-starter > > > Ok, I feared that class was one where the size is critical yeah... > > > - I don't like the inconsistency where all addresses are demangled in the > > DWARF code, except the line tables, which are in the "generic" code > > Yup. The reason for doing it like this was as I tried to find the most narrow > interface where I could catch all of them. Is there any corresponding place in generic code where one could do the same operation on the addresses that comes from pc_lo/pc_hi ranges from .debug_info and .debug_aranges (not familiar with this section and when it's generated etc), if that would end up touching fewer places? The existing predecent in DWARFCallFrameInfo::GetFDEIndex is dwarf specific, but in generic code outside of the dwarf symbolfile plugin. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70840/new/ https://reviews.llvm.org/D70840 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits