[llvm-commits] [llvm] r129233 - /llvm/trunk/include/llvm/ADT/DenseMapInfo.h

Chris Lattner sabre at nondot.org
Sat Apr 9 14:20:23 PDT 2011


Author: lattner
Date: Sat Apr  9 16:20:23 2011
New Revision: 129233

URL: http://llvm.org/viewvc/llvm-project?rev=129233&view=rev
Log:
drive by bug fix: DenseMapInfo::isEqual should be determined according to
isEqual of its members, not operator==.

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

Modified: llvm/trunk/include/llvm/ADT/DenseMapInfo.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/ADT/DenseMapInfo.h?rev=129233&r1=129232&r2=129233&view=diff
==============================================================================
--- llvm/trunk/include/llvm/ADT/DenseMapInfo.h (original)
+++ llvm/trunk/include/llvm/ADT/DenseMapInfo.h Sat Apr  9 16:20:23 2011
@@ -157,7 +157,10 @@
     key ^= (key >> 31);
     return (unsigned)key;
   }
-  static bool isEqual(const Pair& LHS, const Pair& RHS) { return LHS == RHS; }
+  static bool isEqual(const Pair &LHS, const Pair &RHS) {
+    return FirstInfo::isEqual(LHS.first, RHS.first) && 
+           SecondInfo::isEqual(LHS.second, RHS.second);
+  }
 };
 
 } // end namespace llvm





More information about the llvm-commits mailing list