[PATCH] D86539: [Debuginfo][llvm-dwarfutil] llvm-dwarfutil dsymutil-like tool for ELF.

[Alexey Lapshin via Phabricator via llvm-commits]

avl added a comment.

> Currently if we rely on tombstone values, then we might end up missing relocations that have a non-zero addend where the base address is the tombstone value. The way that llvm-gsymutil (GSYM symbol format that can convert DWARF into GSYM) does it is to check for all executable section ranges and only accept any DW_TAG_subprogram values form these ranges.

I think we could do this for the --tombstone=bfd case which assumed to be compatible with old behavior.

according to https://reviews.llvm.org/D84825, starting from LLD 11 the pattern "0+addend" should not be used by linker. 
So we can use -1/-2 only for --tombstone=maxpc and do that additional check for DW_TAG_subprogram range for --tombstone=bfd.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86539