diff options
| author | Nick Lewycky <nicholas@mxc.ca> | 2008-03-09 08:50:23 +0000 |
|---|---|---|
| committer | Nick Lewycky <nicholas@mxc.ca> | 2008-03-09 08:50:23 +0000 |
| commit | 271506f29c808545b33013076d03e753e401fa2d (patch) | |
| tree | 3cbe912f7d01dcfe85c51c7719ac8b4af79fd90e /llvm/lib/Transforms | |
| parent | 9e07537e8cfdacb419c8c08d19b1893c6e0ceaa6 (diff) | |
| download | bcm5719-llvm-271506f29c808545b33013076d03e753e401fa2d.tar.gz bcm5719-llvm-271506f29c808545b33013076d03e753e401fa2d.zip | |
Don't eliminate blocks that are only reachable by unwind_to.
llvm-svn: 48106
Diffstat (limited to 'llvm/lib/Transforms')
| -rw-r--r-- | llvm/lib/Transforms/Scalar/InstructionCombining.cpp | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/Scalar/InstructionCombining.cpp b/llvm/lib/Transforms/Scalar/InstructionCombining.cpp index 8e99dcc7db2..6783607e370 100644 --- a/llvm/lib/Transforms/Scalar/InstructionCombining.cpp +++ b/llvm/lib/Transforms/Scalar/InstructionCombining.cpp @@ -10796,11 +10796,15 @@ static void AddReachableCodeToWorklist(BasicBlock *BB, // Recursively visit successors. If this is a branch or switch on a // constant, only visit the reachable successor. + if (BB->getUnwindDest()) + Worklist.push_back(BB->getUnwindDest()); TerminatorInst *TI = BB->getTerminator(); if (BranchInst *BI = dyn_cast<BranchInst>(TI)) { if (BI->isConditional() && isa<ConstantInt>(BI->getCondition())) { bool CondVal = cast<ConstantInt>(BI->getCondition())->getZExtValue(); - Worklist.push_back(BI->getSuccessor(!CondVal)); + BasicBlock *ReachableBB = BI->getSuccessor(!CondVal); + if (ReachableBB != BB->getUnwindDest()) + Worklist.push_back(ReachableBB); continue; } } else if (SwitchInst *SI = dyn_cast<SwitchInst>(TI)) { @@ -10808,7 +10812,9 @@ static void AddReachableCodeToWorklist(BasicBlock *BB, // See if this is an explicit destination. for (unsigned i = 1, e = SI->getNumSuccessors(); i != e; ++i) if (SI->getCaseValue(i) == Cond) { - Worklist.push_back(SI->getSuccessor(i)); + BasicBlock *ReachableBB = SI->getSuccessor(i); + if (ReachableBB != BB->getUnwindDest()) + Worklist.push_back(ReachableBB); continue; } |

