[PATCH] D95617: [DWARF] Inlined variables with no location should not have a DW_TAG_variable

Rainer Orth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 8 12:31:33 PST 2022


ro added a comment.

In D95617#3305333 <https://reviews.llvm.org/D95617#3305333>, @dblaikie wrote:

> 



> Did the add instructions get somehow lost in Sparc? (like they got merged into the ret instruction somehow, or something in the Sparc backend transformed the add instructions and lost the debug locations along the way? Perhaps that could/should be fixed - but also, I think using `call` instructions rather than `add` instructions is more robust for this sort of testing (call to an opaque/declared-but-not-defined function is unlikely to get transformed in interesting ways or lose its debug location), so the test case could be changed to do that)

The `add` insns didn't get lost at all, but were placed in the delay slots of `retl`.  Which reminded me: this seems to be yet another instance of Issue #46473.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95617



More information about the llvm-commits mailing list