[llvm-commits] [llvm] r95127 - /llvm/trunk/include/llvm/ADT/DenseSet.h

Dan Gohman gohman at apple.com
Tue Feb 2 13:11:22 PST 2010


Author: djg
Date: Tue Feb  2 15:11:22 2010
New Revision: 95127

URL: http://llvm.org/viewvc/llvm-project?rev=95127&view=rev
Log:
Make DenseSet's erase pass on the return value rather than swallowing it.

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

Modified: llvm/trunk/include/llvm/ADT/DenseSet.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/ADT/DenseSet.h?rev=95127&r1=95126&r2=95127&view=diff

==============================================================================
--- llvm/trunk/include/llvm/ADT/DenseSet.h (original)
+++ llvm/trunk/include/llvm/ADT/DenseSet.h Tue Feb  2 15:11:22 2010
@@ -41,8 +41,8 @@
     return TheMap.count(V);
   }
 
-  void erase(const ValueT &V) {
-    TheMap.erase(V);
+  bool erase(const ValueT &V) {
+    return TheMap.erase(V);
   }
 
   DenseSet &operator=(const DenseSet &RHS) {





More information about the llvm-commits mailing list