diff options
| author | Adrian Prantl <aprantl@apple.com> | 2018-09-14 19:38:45 +0000 |
|---|---|---|
| committer | Adrian Prantl <aprantl@apple.com> | 2018-09-14 19:38:45 +0000 |
| commit | 16f58d18506bf3c0bf89cf24e3fea88d5a8154dc (patch) | |
| tree | 69eaf7a8a90a6cff82db9467a4f9d80e694f4f54 /llvm/lib/CodeGen | |
| parent | ec53c89fde3aff5247cac753c84629d07507b418 (diff) | |
| download | bcm5719-llvm-16f58d18506bf3c0bf89cf24e3fea88d5a8154dc.tar.gz bcm5719-llvm-16f58d18506bf3c0bf89cf24e3fea88d5a8154dc.zip | |
Fix debug info for SelectionDAG legalization of DAG nodes with two results.
This patch fixes the debug info handling for SelectionDAG legalization
of DAG nodes with two results. When an replaced SDNode has more than
one result, transferDbgValues was always copying the SDDbgValue from
the first result and attaching them to all members. In reality
SelectionDAG::ReplaceAllUsesWith() is given an array of SDNodes
(though the type signature doesn't make this obvious (cf. the call
site code in ReplaceNode()).
rdar://problem/44162227
Differential Revision: https://reviews.llvm.org/D52112
llvm-svn: 342264
Diffstat (limited to 'llvm/lib/CodeGen')
| -rw-r--r-- | llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp b/llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp index 4a106c2599e..afe7bfbf9f5 100644 --- a/llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp @@ -7797,7 +7797,7 @@ void SelectionDAG::ReplaceAllUsesWith(SDNode *From, const SDValue *To) { // Preserve Debug Info. for (unsigned i = 0, e = From->getNumValues(); i != e; ++i) - transferDbgValues(SDValue(From, i), *To); + transferDbgValues(SDValue(From, i), To[i]); // Iterate over just the existing users of From. See the comments in // the ReplaceAllUsesWith above. |
