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

[Alexander Yermolovich via Phabricator via llvm-commits]

ayermolo added a comment.

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

> In D137139#4078544 <https://reviews.llvm.org/D137139#4078544>, @ayermolo wrote:
>
>> Well this completely fell through the cracks. :facepalm 
>> @dblaikie do you want to be an official reviewer for this?
>
> Sure, don't mind formal or informal.
>
> Was waiting on a few questions I asked @clayborg in my last comment. (perhaps the most practical one: Perhaps we could address this issue by removing support for aranges+gmlt instead of fixing this bug, as a step towards deprecating/removing aranges - though I'm not totally opposed to fixing the bug, just somewhat averse to supporting this use case where I think a bit more pressure on folks still using it to instead migrate to CU ranges might be good)

Thank you. I'll ping Greg if he hasn't replied when he is back.
As for removing aranges, one of our internal symbolization tools relies on aranges currently. I don't think it's in the cards for that to change in short/medium future. :(


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137139