diff options
author | Devang Patel <dpatel@apple.com> | 2008-07-03 06:48:21 +0000 |
---|---|---|
committer | Devang Patel <dpatel@apple.com> | 2008-07-03 06:48:21 +0000 |
commit | 4be7d294f8045f08bcdbd74f1886ea8121a77022 (patch) | |
tree | aee28bb3644fa5c4877b9d4e62a1044814f7b124 | |
parent | b76247b8b1534f42d6fcaa2290c8e4e1baf5443a (diff) | |
download | external_llvm-4be7d294f8045f08bcdbd74f1886ea8121a77022.zip external_llvm-4be7d294f8045f08bcdbd74f1886ea8121a77022.tar.gz external_llvm-4be7d294f8045f08bcdbd74f1886ea8121a77022.tar.bz2 |
Reconstruct dom info, if loop is unswitched.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@53086 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/Transforms/Scalar/LoopUnswitch.cpp | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/lib/Transforms/Scalar/LoopUnswitch.cpp b/lib/Transforms/Scalar/LoopUnswitch.cpp index f57b7bb..02dac92 100644 --- a/lib/Transforms/Scalar/LoopUnswitch.cpp +++ b/lib/Transforms/Scalar/LoopUnswitch.cpp @@ -113,8 +113,8 @@ namespace { AU.addRequiredID(LCSSAID); AU.addPreservedID(LCSSAID); // FIXME: Loop Unswitch does not preserve dominator info in all cases. - // AU.addPreserved<DominatorTree>(); - // AU.addPreserved<DominanceFrontier>(); + AU.addPreserved<DominatorTree>(); + AU.addPreserved<DominanceFrontier>(); } private: @@ -206,7 +206,6 @@ bool LoopUnswitch::runOnLoop(Loop *L, LPPassManager &LPM_Ref) { DT = getAnalysisToUpdate<DominatorTree>(); currentLoop = L; bool Changed = false; - do { assert(currentLoop->isLCSSAForm()); redoLoop = false; @@ -459,7 +458,13 @@ bool LoopUnswitch::UnswitchIfProfitable(Value *LoopCond, Constant *Val){ } else { UnswitchNontrivialCondition(LoopCond, Val, currentLoop); } - + + // FIXME: Reconstruct dom info, because it is not preserved properly. + Function *F = loopHeader->getParent(); + if (DT) + DT->runOnFunction(*F); + if (DF) + DF->runOnFunction(*F); return true; } |