tidy up and delete a dead smallvector.
authorChris Lattner <sabre@nondot.org>
Mon, 28 Dec 2009 19:49:00 +0000 (19:49 +0000)
committerChris Lattner <sabre@nondot.org>
Mon, 28 Dec 2009 19:49:00 +0000 (19:49 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@92223 91177308-0d34-0410-b5e6-96231b3b80d8

lib/VMCore/Metadata.cpp

index 0a3ddcbbe1b6361cde546a7baa7ac77c57660d5e..3d136ccf3c401dffc6eff4b68422e8d4343c5e3c 100644 (file)
 #include "llvm/Support/ValueHandle.h"
 using namespace llvm;
 
-//===----------------------------------------------------------------------===//
-// MetadataBase implementation.
-//
-
 //===----------------------------------------------------------------------===//
 // MDString implementation.
 //
@@ -130,6 +126,8 @@ MDNode *MDNode::get(LLVMContext &Context, Value*const* Vals, unsigned NumVals,
 void MDNode::Profile(FoldingSetNodeID &ID) const {
   for (unsigned i = 0, e = getNumElements(); i != e; ++i)
     ID.AddPointer(getElement(i));
+  // HASH TABLE COLLISIONS?
+  // DO NOT REINSERT AFTER AN OPERAND DROPS TO NULL!
 }
 
 
@@ -433,10 +431,8 @@ void MetadataContextImpl::ValueIsCloned(const Instruction *In1,
   MDStoreTy::iterator I = MetadataStore.find(In1);
   assert(I != MetadataStore.end() && "Invalid custom metadata info!");
 
-  // FIXME : Give all metadata handlers a chance to adjust.
-
+  // FIXME: Give all metadata handlers a chance to adjust.
   MDMapTy &In1Info = I->second;
-  MDMapTy In2Info;
   for (MDMapTy::iterator I = In1Info.begin(), E = In1Info.end(); I != E; ++I)
     addMD(I->first, I->second, In2);
 }
@@ -449,15 +445,14 @@ void MetadataContextImpl::ValueIsRAUWd(Value *V1, Value *V2) {
   if (!I1 || !I2)
     return;
 
-  // FIXME : Give custom handlers a chance to override this.
+  // FIXME: Give custom handlers a chance to override this.
   ValueIsCloned(I1, I2);
 }
 
 //===----------------------------------------------------------------------===//
 // MetadataContext implementation.
 //
-MetadataContext::MetadataContext() 
-  : pImpl(new MetadataContextImpl()) { }
+MetadataContext::MetadataContext() : pImpl(new MetadataContextImpl()) { }
 MetadataContext::~MetadataContext() { delete pImpl; }
 
 /// isValidName - Return true if Name is a valid custom metadata handler name.