diff options
author | Dan Gohman <gohman@apple.com> | 2010-09-28 22:07:19 +0000 |
---|---|---|
committer | Dan Gohman <gohman@apple.com> | 2010-09-28 22:07:19 +0000 |
commit | f53458f693fcf813b4c6c785034141017bf35893 (patch) | |
tree | d7a74d1b2b56d3cc6ca21048ef8c44a6f7517e44 /lib | |
parent | 654d5440a477b1f6c89b051107e041a331f78e27 (diff) | |
download | llvm-f53458f693fcf813b4c6c785034141017bf35893.tar.gz llvm-f53458f693fcf813b4c6c785034141017bf35893.tar.bz2 llvm-f53458f693fcf813b4c6c785034141017bf35893.tar.xz |
When an MDNode changes to become identical to another MDNode,
delete the MDNode that changed, rather than the other MDNode.
This is less work, because it doesn't require the changed node
to be re-inserted into the uniquing map and it doesn't require
the is-function-local flag to be recomputed. Also, it avoids
trouble when the existing node is part of a complicated
data structure.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@114996 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib')
-rw-r--r-- | lib/VMCore/Metadata.cpp | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/lib/VMCore/Metadata.cpp b/lib/VMCore/Metadata.cpp index 162884c5f6..0b8e8dfa8b 100644 --- a/lib/VMCore/Metadata.cpp +++ b/lib/VMCore/Metadata.cpp @@ -339,15 +339,14 @@ void MDNode::replaceOperand(MDNodeOperand *Op, Value *To) { // Now that the node is out of the folding set, get ready to reinsert it. // First, check to see if another node with the same operands already exists - // in the set. If it doesn't exist, this returns the position to insert it. + // in the set. If so, then this node is redundant. FoldingSetNodeID ID; Profile(ID); void *InsertPoint; if (MDNode *N = pImpl->MDNodeSet.FindNodeOrInsertPos(ID, InsertPoint)) { - N->replaceAllUsesWith(this); - N->destroy(); - N = pImpl->MDNodeSet.FindNodeOrInsertPos(ID, InsertPoint); - assert(N == 0 && "shouldn't be in the map now!"); (void)N; + replaceAllUsesWith(N); + destroy(); + return; } // InsertPoint will have been set by the FindNodeOrInsertPos call. |