[PATCH] D53469: [DWARF] Use a function-local offset for AT_call_return_pc

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 21 13:41:33 PST 2019


dblaikie added a comment.

(we really shouldn't have any abstraction that allows creating a DW_FORM_addr unconditionally - since that's not a valid form in Split DWARF, it should always go through something that adds the address to the address pool and uses FORM_addrx in Split DWARF at least (& probably uses addrx (or something more general like the hypothetical addr_offset, or a constant relative (relative to what, as discussed in my previous comment) encoding, etc) - which is llvm::DwarfDebug::addLabelAddress)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D53469





More information about the llvm-commits mailing list