Do one map lookup instead of two.
authorDan Gohman <gohman@apple.com>
Tue, 25 May 2010 21:59:42 +0000 (21:59 +0000)
committerDan Gohman <gohman@apple.com>
Tue, 25 May 2010 21:59:42 +0000 (21:59 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@104645 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/SelectionDAG/FastISel.cpp

index 1996c192ec7120041e06c41f7c1988824dcfbbe8..95f4d072e01f16f5a6e5c5963562ab481563e3c0 100644 (file)
@@ -99,8 +99,9 @@ unsigned FastISel::getRegForValue(const Value *V) {
   // cache values defined by Instructions across blocks, and other values
   // only locally. This is because Instructions already have the SSA
   // def-dominates-use requirement enforced.
-  if (ValueMap.count(V))
-    return ValueMap[V];
+  DenseMap<const Value *, unsigned>::iterator I = ValueMap.find(V);
+  if (I != ValueMap.end())
+    return I->second;
   unsigned Reg = LocalValueMap[V];
   if (Reg != 0)
     return Reg;