[llvm-commits] [llvm] r81355 - /llvm/trunk/lib/VMCore/Metadata.cpp

Devang Patel dpatel at apple.com
Wed Sep 9 10:30:05 PDT 2009


Author: dpatel
Date: Wed Sep  9 12:30:04 2009
New Revision: 81355

URL: http://llvm.org/viewvc/llvm-project?rev=81355&view=rev
Log:
Enable MDNode uniquing.

Modified:
    llvm/trunk/lib/VMCore/Metadata.cpp

Modified: llvm/trunk/lib/VMCore/Metadata.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/VMCore/Metadata.cpp?rev=81355&r1=81354&r2=81355&view=diff

==============================================================================
--- llvm/trunk/lib/VMCore/Metadata.cpp (original)
+++ llvm/trunk/lib/VMCore/Metadata.cpp Wed Sep  9 12:30:04 2009
@@ -90,16 +90,11 @@
   for (unsigned i = 0; i != NumVals; ++i)
     ID.AddPointer(Vals[i]);
 
-  // FIXME: MDNode uniquing disabled temporarily.
-#ifndef ENABLE_MDNODE_UNIQUING
-  return new MDNode(Context, Vals, NumVals);
-#endif
-
   pImpl->ConstantsLock.reader_acquire();
   void *InsertPoint;
   MDNode *N = pImpl->MDNodeSet.FindNodeOrInsertPos(ID, InsertPoint);
   pImpl->ConstantsLock.reader_release();
-
+  
   if (!N) {
     sys::SmartScopedWriter<true> Writer(pImpl->ConstantsLock);
     N = pImpl->MDNodeSet.FindNodeOrInsertPos(ID, InsertPoint);
@@ -120,27 +115,12 @@
 }
 
 MDNode::~MDNode() {
-  // FIXME: MDNode uniquing disabled temporarily.
-#ifdef ENABLE_MDNODE_UNIQUING
   getType()->getContext().pImpl->MDNodeSet.RemoveNode(this);
-#endif
   dropAllReferences();
 }
 
 // Replace value from this node's element list.
 void MDNode::replaceElement(Value *From, Value *To) {
-  // FIXME: MDNode uniquing disabled temporarily.
-#ifndef ENABLE_MDNODE_UNIQUING
-  if (From == To || !getType())
-    return;
-
-  for (SmallVector<ElementVH, 4>::iterator I = Node.begin(),
-         E = Node.end(); I != E; ++I)
-    if (*I && *I == From)
-      *I = ElementVH(To, this);
-  return;
-#endif
-
   if (From == To || !getType())
     return;
   LLVMContext &Context = getType()->getContext();





More information about the llvm-commits mailing list