[llvm] r192602 - Basic blocks typically have few predecessors. Use a SmallDenseMap to

Chris Lattner sabre at nondot.org
Mon Oct 14 09:05:55 PDT 2013


Author: lattner
Date: Mon Oct 14 11:05:55 2013
New Revision: 192602

URL: http://llvm.org/viewvc/llvm-project?rev=192602&view=rev
Log:
Basic blocks typically have few predecessors.  Use a SmallDenseMap to
avoid a heap allocation when this is the case.

Modified:
    llvm/trunk/lib/Transforms/Utils/SSAUpdater.cpp

Modified: llvm/trunk/lib/Transforms/Utils/SSAUpdater.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Utils/SSAUpdater.cpp?rev=192602&r1=192601&r2=192602&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Utils/SSAUpdater.cpp (original)
+++ llvm/trunk/lib/Transforms/Utils/SSAUpdater.cpp Mon Oct 14 11:05:55 2013
@@ -63,7 +63,7 @@ void SSAUpdater::AddAvailableValue(Basic
 }
 
 static bool IsEquivalentPHI(PHINode *PHI,
-                            DenseMap<BasicBlock*, Value*> &ValueMapping) {
+                          SmallDenseMap<BasicBlock*, Value*, 8> &ValueMapping) {
   unsigned PHINumValues = PHI->getNumIncomingValues();
   if (PHINumValues != ValueMapping.size())
     return false;
@@ -136,8 +136,8 @@ Value *SSAUpdater::GetValueInMiddleOfBlo
   // Otherwise, we do need a PHI: check to see if we already have one available
   // in this block that produces the right value.
   if (isa<PHINode>(BB->begin())) {
-    DenseMap<BasicBlock*, Value*> ValueMapping(PredValues.begin(),
-                                               PredValues.end());
+    SmallDenseMap<BasicBlock*, Value*, 8> ValueMapping(PredValues.begin(),
+                                                       PredValues.end());
     PHINode *SomePHI;
     for (BasicBlock::iterator It = BB->begin();
          (SomePHI = dyn_cast<PHINode>(It)); ++It) {





More information about the llvm-commits mailing list