diff options
| author | Benjamin Kramer <benny.kra@googlemail.com> | 2012-05-27 22:53:10 +0000 |
|---|---|---|
| committer | Benjamin Kramer <benny.kra@googlemail.com> | 2012-05-27 22:53:10 +0000 |
| commit | 8e337120133c746640246feb9383556d383a94be (patch) | |
| tree | db6f775cb7ebce6093611417e25a1583a37301e7 | |
| parent | 65195411ccb18ea8327d3eabdfa980eaf535d929 (diff) | |
DenseMap: Use an early exit when there is nothing to do in DestroyAll().
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@157550 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | include/llvm/ADT/DenseMap.h | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/include/llvm/ADT/DenseMap.h b/include/llvm/ADT/DenseMap.h index 1fc0eed706..0ceca926c6 100644 --- a/include/llvm/ADT/DenseMap.h +++ b/include/llvm/ADT/DenseMap.h @@ -273,6 +273,9 @@ public: private: void DestroyAll() { + if (NumBuckets == 0) // Nothing to do. + return; + const KeyT EmptyKey = getEmptyKey(), TombstoneKey = getTombstoneKey(); for (BucketT *P = Buckets, *E = Buckets+NumBuckets; P != E; ++P) { if (!KeyInfoT::isEqual(P->first, EmptyKey) && @@ -281,12 +284,10 @@ private: P->first.~KeyT(); } - if (NumBuckets) { #ifndef NDEBUG - memset((void*)Buckets, 0x5a, sizeof(BucketT)*NumBuckets); + memset((void*)Buckets, 0x5a, sizeof(BucketT)*NumBuckets); #endif - operator delete(Buckets); - } + operator delete(Buckets); } void CopyFrom(const DenseMap& other) { |