[PATCH] D96827: [DWARF] Check for AddrOffsetSectionBase to work with DWO Units.

Alexander Yermolovich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 12 13:48:33 PST 2021


ayermolo added a comment.

In D96827#2623393 <https://reviews.llvm.org/D96827#2623393>, @dblaikie wrote:

> Looks good - just a couple of minor fixes to make before committing.
>
> (I assume by your patch that this is only reproducible with a linked binary? please confirm that (in a comment here) before committing)

Sorry for not replying. I was trying to see if I could fix the build failure. It works on my CentOS machine.
Yes I think it's only reproducible with the binary/object combo. You need to have .debug_addr to be in different file than DWO CU. The case where they are all in same file is covered by LLC, Case 1, and works.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96827



More information about the llvm-commits mailing list