diff options
author | Dan Gohman <gohman@apple.com> | 2010-09-01 14:00:35 +0000 |
---|---|---|
committer | Dan Gohman <gohman@apple.com> | 2010-09-01 14:00:35 +0000 |
commit | e3955df639ff9aff990f628ef6a219ff5efdbc81 (patch) | |
tree | 59fc80421919816021622260f4ae70d1ea434c12 /include/llvm/ADT/DenseMap.h | |
parent | f8ff40c0598c6d8707a639ec63644e5e5496678a (diff) | |
download | llvm-e3955df639ff9aff990f628ef6a219ff5efdbc81.tar.gz llvm-e3955df639ff9aff990f628ef6a219ff5efdbc81.tar.bz2 llvm-e3955df639ff9aff990f628ef6a219ff5efdbc81.tar.xz |
Make the iterator form of erase return void, since it always succeeds,
and since this is what std::map and std::set do.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@112701 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'include/llvm/ADT/DenseMap.h')
-rw-r--r-- | include/llvm/ADT/DenseMap.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/include/llvm/ADT/DenseMap.h b/include/llvm/ADT/DenseMap.h index c53e255e1c..06a1575da4 100644 --- a/include/llvm/ADT/DenseMap.h +++ b/include/llvm/ADT/DenseMap.h @@ -185,13 +185,12 @@ public: ++NumTombstones; return true; } - bool erase(iterator I) { + void erase(iterator I) { BucketT *TheBucket = &*I; TheBucket->second.~ValueT(); TheBucket->first = getTombstoneKey(); --NumEntries; ++NumTombstones; - return true; } void swap(DenseMap& RHS) { |