diff options
| author | Frederic Riss <friss@apple.com> | 2014-09-15 07:50:42 +0000 |
|---|---|---|
| committer | Frederic Riss <friss@apple.com> | 2014-09-15 07:50:42 +0000 |
| commit | dd7aec5ddec1881c1d9c7ba7b26e1bc3abe19746 (patch) | |
| tree | e83a38032aad071d20e4742ca7ef228585fdcd54 | |
| parent | 36acf0fb8b8627bd586fe7df8872b1c8a17ab7cd (diff) | |
| download | bcm5719-llvm-dd7aec5ddec1881c1d9c7ba7b26e1bc3abe19746.tar.gz bcm5719-llvm-dd7aec5ddec1881c1d9c7ba7b26e1bc3abe19746.zip | |
Fix DebugInfo replaceAllUsesWith.
Summary:
replaceAllUsesWith had been modified to allow a DbgNode value to be
replaced by itself. In that case a new node is created by copying the
current DbgNode and the copy is used as replacement value.
When that copying happens, the value stored in this->DbgNode at the end
of RAUW would be a reference to the Node that has just been deleted.
This doesn't produce any bug right now, because the DI node on which we
call RAUW won't be used again.
Reviewers: dblaikie, echristo, aprantl
Subscribers: llvm-commits
Differential Revision: http://reviews.llvm.org/D5326
llvm-svn: 217749
| -rw-r--r-- | llvm/lib/IR/DebugInfo.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/IR/DebugInfo.cpp b/llvm/lib/IR/DebugInfo.cpp index 71c32fc969d..4274d82d158 100644 --- a/llvm/lib/IR/DebugInfo.cpp +++ b/llvm/lib/IR/DebugInfo.cpp @@ -384,7 +384,7 @@ void DIDescriptor::replaceAllUsesWith(LLVMContext &VMContext, DIDescriptor D) { const Value *V = cast_or_null<Value>(DN); Node->replaceAllUsesWith(const_cast<Value *>(V)); MDNode::deleteTemporary(Node); - DbgNode = D; + DbgNode = DN; } /// replaceAllUsesWith - Replace all uses of the MDNode used by this |
