From f51825e3fb74db0e6c85583a9a662000eb1e8388 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Mon, 6 May 2002 17:54:10 +0000 Subject: [PATCH] * Use simplified interface to constant propogation stuff. * Remove dead PHI case (which could not work due to getNumOperands rather than getNumIncomingValues. This really belongs in InstCombine, anyway so we'll move it there. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@2497 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/Scalar/ConstantProp.cpp | 43 +++++--------------------- 1 file changed, 8 insertions(+), 35 deletions(-) diff --git a/lib/Transforms/Scalar/ConstantProp.cpp b/lib/Transforms/Scalar/ConstantProp.cpp index 9e519e7cf77..aa46f2bb49a 100644 --- a/lib/Transforms/Scalar/ConstantProp.cpp +++ b/lib/Transforms/Scalar/ConstantProp.cpp @@ -179,43 +179,16 @@ bool ConstantFoldTerminator(BasicBlock *BB, BasicBlock::iterator &II, // bool doConstantPropogation(BasicBlock *BB, BasicBlock::iterator &II) { Instruction *Inst = *II; - if (BinaryOperator *BO = dyn_cast(Inst)) { - Constant *D1 = dyn_cast(BO->getOperand(0)); - Constant *D2 = dyn_cast(BO->getOperand(1)); - - if (D1 && D2) - return ConstantFoldBinaryInst(BB, II, BO, D1, D2); - - } else if (CastInst *CI = dyn_cast(Inst)) { - Constant *D = dyn_cast(CI->getOperand(0)); - if (D) return ConstantFoldCast(BB, II, CI, D); - - } else if (UnaryOperator *UInst = dyn_cast(Inst)) { - Constant *D = dyn_cast(UInst->getOperand(0)); - if (D) return ConstantFoldUnaryInst(BB, II, UInst, D); - } else if (TerminatorInst *TInst = dyn_cast(Inst)) { + if (TerminatorInst *TInst = dyn_cast(Inst)) { return ConstantFoldTerminator(BB, II, TInst); + } else if (Constant *C = ConstantFoldInstruction(Inst)) { + // Replaces all of the uses of a variable with uses of the constant. + Inst->replaceAllUsesWith(C); + + // Remove the instruction from the basic block... + delete BB->getInstList().remove(II); + return true; - } else if (PHINode *PN = dyn_cast(Inst)) { - // If it's a PHI node and only has one operand - // Then replace it directly with that operand. - assert(PN->getNumOperands() && "PHI Node must have at least one operand!"); - if (PN->getNumOperands() == 1) { // If the PHI Node has exactly 1 operand - Value *V = PN->getOperand(0); - PN->replaceAllUsesWith(V); // Replace all uses of this PHI - // Unlink from basic block - PN->getParent()->getInstList().remove(II); - if (PN->hasName()) // Inherit PHINode name - V->setName(PN->getName(), BB->getParent()->getSymbolTableSure()); - delete PN; // Finally, delete the node... - return true; - } - } else if (ShiftInst *SI = dyn_cast(Inst)) { - Constant *D1 = dyn_cast(SI->getOperand(0)); - Constant *D2 = dyn_cast(SI->getOperand(1)); - - if (D1 && D2) - return ConstantFoldShiftInst(BB, II, SI, D1, D2); } return false;