From 1d880e51142566838131b091306dd03549adb6d2 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Mon, 28 Dec 2009 09:10:16 +0000 Subject: [PATCH] avoid temporary CallbackVH's. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@92218 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/VMCore/Metadata.cpp | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/lib/VMCore/Metadata.cpp b/lib/VMCore/Metadata.cpp index c1213a3d3a9..2bf2c60f7ce 100644 --- a/lib/VMCore/Metadata.cpp +++ b/lib/VMCore/Metadata.cpp @@ -64,6 +64,11 @@ public: MDNodeElement(Value *V, MDNode *P) : CallbackVH(V), Parent(P) {} ~MDNodeElement() {} + void set(Value *V, MDNode *P) { + setValPtr(V); + Parent = P; + } + virtual void deleted(); virtual void allUsesReplacedWith(Value *NV); }; @@ -91,7 +96,7 @@ MDNode::MDNode(LLVMContext &C, Value *const *Vals, unsigned NumVals, Operands = new MDNodeElement[NumOperands]; MDNodeElement *Ptr = Operands; for (unsigned i = 0; i != NumVals; ++i) - Ptr[i] = MDNodeElement(Vals[i], this); + Ptr[i].set(Vals[i], this); if (isFunctionLocal) SubclassData |= FunctionLocalBit; @@ -161,9 +166,8 @@ void MDNode::replaceElement(Value *From, Value *To) { // Replace From element(s) in place. for (SmallVector::iterator I = Indexes.begin(), E = Indexes.end(); - I != E; ++I) { - Operands[*I] = MDNodeElement(To, this); - } + I != E; ++I) + Operands[*I].set(To, this); // Insert updated "this" into the context's folding node set. // If a node with same element list already exist then before inserting