diff options
| author | Josh Stone <jistone@redhat.com> | 2018-09-11 17:52:01 +0000 |
|---|---|---|
| committer | Josh Stone <jistone@redhat.com> | 2018-09-11 17:52:01 +0000 |
| commit | f446facab0e38c7496ac581f31335e60ec896cb9 (patch) | |
| tree | 91b4d1fb0782961bc25c1d819c21a61be6124bd5 /llvm/lib/CodeGen/MachineInstr.cpp | |
| parent | 8b7b743cc1b83eec0f43dab625dff9b3817a5a3a (diff) | |
| download | llvm-f446facab0e38c7496ac581f31335e60ec896cb9.zip llvm-f446facab0e38c7496ac581f31335e60ec896cb9.tar.gz llvm-f446facab0e38c7496ac581f31335e60ec896cb9.tar.bz2 | |
[GlobalISel] Lower dbg.declare into indirect DBG_VALUE
Summary:
D31439 changed the semantics of dbg.declare to take the address of a
variable as the first argument, making it indirect. It specifically
updated FastISel for this change here:
https://reviews.llvm.org/D31439#change-WVArzi177jPl
GlobalISel needs to follow suit, or else it will be missing a level of
indirection in the generated debuginfo. This problem was seen in a Rust
debuginfo test on aarch64, since GlobalISel is used at -O0 for aarch64.
https://github.com/rust-lang/rust/issues/49807
https://bugzilla.redhat.com/show_bug.cgi?id=1611597
https://bugzilla.redhat.com/show_bug.cgi?id=1625768
Reviewers: dblaikie, aprantl, t.p.northover, javed.absar, rnk
Reviewed By: rnk
Subscribers: #debug-info, rovka, kristof.beyls, JDevlieghere, llvm-commits, tstellar
Differential Revision: https://reviews.llvm.org/D51749
llvm-svn: 341969
Diffstat (limited to 'llvm/lib/CodeGen/MachineInstr.cpp')
0 files changed, 0 insertions, 0 deletions