[PATCH] D137139: [DWARF] Fix handling of .debug_aranges with -g1

[Alexander Yermolovich via Phabricator via llvm-commits]

ayermolo added a comment.

In D137139#3899179 <https://reviews.llvm.org/D137139#3899179>, @dblaikie wrote:

> This is the sort of thing I kind of wanted to avoid when we were talking about adding the aranges verifier - fixing bugs in aranges when instead we should be fixing bugs in consumers to not depend on aranges.
>
> But anyway - if you really want/need aranges, perhaps a deeper fix would be good? We could emit aranges as CU ranges + data ranges instead of keeping an entirely separate list that can get out of sync? (& while we're at it, probably move the data ranges to the CU - rather than having to map them back from another independent data structure in DwarfDebug?)

I totally agree... Trying to see if we can migrate internal tool from using aranges. Although considering this is a bit of a "special case" with -g1, might be OK just let it be for that tool.

You have a good point. @clayborg can we just change lldb to ignore this section?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137139/new/

https://reviews.llvm.org/D137139