X-Git-Url: http://plrg.eecs.uci.edu/git/?a=blobdiff_plain;f=lib%2FTransforms%2FScalar%2FCondPropagate.cpp;h=7d5e581b22ebf20958c7ab0e936bbcfa843d0e79;hb=f344d83d3956255382b43690a147e1ce26b9fcd8;hp=000ef60a0db863dedcb4391cb79f0f2725084b60;hpb=39c873e38f3b173b51bd9d3d3b4ec1381b829754;p=oota-llvm.git diff --git a/lib/Transforms/Scalar/CondPropagate.cpp b/lib/Transforms/Scalar/CondPropagate.cpp index 000ef60a0db..7d5e581b22e 100644 --- a/lib/Transforms/Scalar/CondPropagate.cpp +++ b/lib/Transforms/Scalar/CondPropagate.cpp @@ -17,6 +17,7 @@ #include "llvm/Constants.h" #include "llvm/Function.h" #include "llvm/Instructions.h" +#include "llvm/IntrinsicInst.h" #include "llvm/Pass.h" #include "llvm/Type.h" #include "llvm/Transforms/Utils/BasicBlockUtils.h" @@ -134,8 +135,8 @@ void CondProp::SimplifyBlock(BasicBlock *BB) { // jump directly to the destination instead of going through this block. void CondProp::SimplifyPredecessors(BranchInst *BI) { // TODO: We currently only handle the most trival case, where the PHI node has - // one use (the branch), and is the only instruction besides the branch in the - // block. + // one use (the branch), and is the only instruction besides the branch and dbg + // intrinsics in the block. PHINode *PN = cast(BI->getCondition()); if (PN->getNumIncomingValues() == 1) { @@ -148,7 +149,12 @@ void CondProp::SimplifyPredecessors(BranchInst *BI) { if (!PN->hasOneUse()) return; BasicBlock *BB = BI->getParent(); - if (!isTerminatorFirstRelevantInsn (BB, BI)) + if (&*BB->begin() != PN) + return; + BasicBlock::iterator BBI = BB->begin(); + BasicBlock::iterator BBE = BB->end(); + while (BBI != BBE && isa(++BBI)) ; + if (&*BBI != BI) return; // Ok, we have this really simple case, walk the PHI operands, looking for @@ -176,13 +182,18 @@ void CondProp::SimplifyPredecessors(BranchInst *BI) { // the destination instead of going through this block. void CondProp::SimplifyPredecessors(SwitchInst *SI) { // TODO: We currently only handle the most trival case, where the PHI node has - // one use (the branch), and is the only instruction besides the branch in the - // block. + // one use (the branch), and is the only instruction besides the branch and + // dbg intrinsics in the block. PHINode *PN = cast(SI->getCondition()); if (!PN->hasOneUse()) return; BasicBlock *BB = SI->getParent(); - if (!isTerminatorFirstRelevantInsn (BB, SI)) + if (&*BB->begin() != PN) + return; + BasicBlock::iterator BBI = BB->begin(); + BasicBlock::iterator BBE = BB->end(); + while (BBI != BBE && isa(++BBI)) ; + if (&*BBI != SI) return; bool RemovedPreds = false;