diff options
author | Cameron Zwarich <zwarich@apple.com> | 2011-01-08 17:01:52 +0000 |
---|---|---|
committer | Cameron Zwarich <zwarich@apple.com> | 2011-01-08 17:01:52 +0000 |
commit | 84986b298ae1110529871a19e1b4c9ab81281e10 (patch) | |
tree | acfb2a73430bb63c7a0e038371a245b35f6724d0 /llvm/lib/Transforms/Utils/Local.cpp | |
parent | 45e6c195d7f48a9507edb4888db2964f1cd928d7 (diff) | |
download | bcm5719-llvm-84986b298ae1110529871a19e1b4c9ab81281e10.tar.gz bcm5719-llvm-84986b298ae1110529871a19e1b4c9ab81281e10.zip |
Make more passes preserve dominators (or state that they preserve dominators if
they all ready do). This removes two dominator recomputations prior to isel,
which is a 1% improvement in total llc time for 403.gcc.
The only potentially suspect thing is making GCStrategy recompute dominators if
it used a custom lowering strategy.
llvm-svn: 123064
Diffstat (limited to 'llvm/lib/Transforms/Utils/Local.cpp')
-rw-r--r-- | llvm/lib/Transforms/Utils/Local.cpp | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/llvm/lib/Transforms/Utils/Local.cpp b/llvm/lib/Transforms/Utils/Local.cpp index 4c2bdbc2266..30e88e976f2 100644 --- a/llvm/lib/Transforms/Utils/Local.cpp +++ b/llvm/lib/Transforms/Utils/Local.cpp @@ -22,6 +22,7 @@ #include "llvm/IntrinsicInst.h" #include "llvm/ADT/DenseMap.h" #include "llvm/ADT/SmallPtrSet.h" +#include "llvm/Analysis/Dominators.h" #include "llvm/Analysis/ConstantFolding.h" #include "llvm/Analysis/InstructionSimplify.h" #include "llvm/Analysis/ProfileInfo.h" @@ -401,6 +402,12 @@ void llvm::MergeBasicBlockIntoOnlyPred(BasicBlock *DestBB, Pass *P) { PredBB->replaceAllUsesWith(DestBB); if (P) { + DominatorTree *DT = P->getAnalysisIfAvailable<DominatorTree>(); + if (DT) { + BasicBlock *PredBBIDom = DT->getNode(PredBB)->getIDom()->getBlock(); + DT->changeImmediateDominator(DestBB, PredBBIDom); + DT->eraseNode(PredBB); + } ProfileInfo *PI = P->getAnalysisIfAvailable<ProfileInfo>(); if (PI) { PI->replaceAllUses(PredBB, DestBB); |