6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2024-08-01 02:29:19 +00:00
Code Issues Projects Releases Wiki Activity

Remove the ValueMap copy constructor. It's not used anywhere,

Browse Source 
and removing it catches the mistake of passing a ValueMap by
copy rather than by reference.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@110549 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Duncan Sands 2010-08-08 12:57:48 +00:00
parent 21d3eeb02b
commit 3850f5ceda
2 changed files with 1 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
include/llvm/ADT/ValueMap.h
View File

@ -82,18 +82,12 @@ class ValueMap {
typedef typename Config::ExtraData ExtraData;
MapT Map;
ExtraData Data;
ValueMap(const ValueMap&); // DO NOT IMPLEMENT
public:
typedef KeyT key_type;
typedef ValueT mapped_type;
typedef std::pair<KeyT, ValueT> value_type;
ValueMap(const ValueMap& Other) : Map(Other.Map), Data(Other.Data) {
// Each ValueMapCVH key contains a pointer to the containing ValueMap.
// The keys in the new map need to point to the new map, not Other.
for (typename MapT::iterator I = Map.begin(), E = Map.end(); I != E; ++I)
I->first.Map = this;
}
explicit ValueMap(unsigned NumInitBuckets = 64)
: Map(NumInitBuckets), Data() {}
explicit ValueMap(const ExtraData &Data, unsigned NumInitBuckets = 64)

9
unittests/ADT/ValueMapTest.cpp
View File

@ -39,15 +39,6 @@ protected:
typedef ::testing::Types<Value, Instruction, const Instruction> KeyTypes;
TYPED_TEST_CASE(ValueMapTest, KeyTypes);
TYPED_TEST(ValueMapTest, CopyConstructor) {
ValueMap<TypeParam*, int> VM1;
VM1[this->AddV.get()] = 7;
ValueMap<TypeParam*, int> VM2(VM1);
this->AddV.reset();
EXPECT_TRUE(VM1.empty());
EXPECT_TRUE(VM2.empty());
}
TYPED_TEST(ValueMapTest, Null) {
ValueMap<TypeParam*, int> VM1;
VM1[NULL] = 7;