[llvm-commits] [llvm] r110577 - /llvm/trunk/include/llvm/ADT/ValueMap.h

Duncan Sands baldrick at free.fr
Mon Aug 9 09:44:56 PDT 2010


Author: baldrick
Date: Mon Aug  9 11:44:56 2010
New Revision: 110577

URL: http://llvm.org/viewvc/llvm-project?rev=110577&view=rev
Log:
Remove the ValueMap operator=, which was wrong (it did't correct the
Map pointers of any contained ValueMapCallbackVH's) and unused.

Modified:
    llvm/trunk/include/llvm/ADT/ValueMap.h

Modified: llvm/trunk/include/llvm/ADT/ValueMap.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/ADT/ValueMap.h?rev=110577&r1=110576&r2=110577&view=diff
==============================================================================
--- llvm/trunk/include/llvm/ADT/ValueMap.h (original)
+++ llvm/trunk/include/llvm/ADT/ValueMap.h Mon Aug  9 11:44:56 2010
@@ -83,6 +83,7 @@
   MapT Map;
   ExtraData Data;
   ValueMap(const ValueMap&); // DO NOT IMPLEMENT
+  ValueMap& operator=(const ValueMap&); // DO NOT IMPLEMENT
 public:
   typedef KeyT key_type;
   typedef ValueT mapped_type;
@@ -160,12 +161,6 @@
     return Map[Wrap(Key)];
   }
 
-  ValueMap& operator=(const ValueMap& Other) {
-    Map = Other.Map;
-    Data = Other.Data;
-    return *this;
-  }
-
   /// isPointerIntoBucketsArray - Return true if the specified pointer points
   /// somewhere into the ValueMap's array of buckets (i.e. either to a key or
   /// value in the ValueMap).





More information about the llvm-commits mailing list