From: Chris Lattner Date: Sun, 5 Aug 2007 08:43:36 +0000 (+0000) Subject: Fix a bug in DenseMap::clear, where we never reset a tombstone X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=7b54452c84e478ab4d49ac08759ca4ec1badf2b2;p=oota-llvm.git Fix a bug in DenseMap::clear, where we never reset a tombstone to EmptyKey. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@40839 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/llvm/ADT/DenseMap.h b/include/llvm/ADT/DenseMap.h index 8b25a82e6ac..fd3f346891b 100644 --- a/include/llvm/ADT/DenseMap.h +++ b/include/llvm/ADT/DenseMap.h @@ -100,10 +100,12 @@ public: const KeyT EmptyKey = getEmptyKey(), TombstoneKey = getTombstoneKey(); for (BucketT *P = Buckets, *E = Buckets+NumBuckets; P != E; ++P) { - if (P->first != EmptyKey && P->first != TombstoneKey) { + if (P->first != EmptyKey) { + if (P->first != TombstoneKey) { + P->second.~ValueT(); + --NumEntries; + } P->first = EmptyKey; - P->second.~ValueT(); - --NumEntries; } } assert(NumEntries == 0 && "Node count imbalance!");