diff options
author | Chris Lattner <sabre@nondot.org> | 2002-05-09 20:11:54 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2002-05-09 20:11:54 +0000 |
commit | 53a5b57737818244515351cf4d147fd2bdd0bc51 (patch) | |
tree | 6e071448a54c4d7a20d86a83d6797860e0342979 | |
parent | 0aa7cd605df9faa57f192c0bbd04af54e7d5461d (diff) |
Handle setcc <global*>, 0 instructions, Global pointers are never null!
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@2582 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/Transforms/Scalar/InstructionCombining.cpp | 22 |
1 files changed, 18 insertions, 4 deletions
diff --git a/lib/Transforms/Scalar/InstructionCombining.cpp b/lib/Transforms/Scalar/InstructionCombining.cpp index 9625398017..385c07004d 100644 --- a/lib/Transforms/Scalar/InstructionCombining.cpp +++ b/lib/Transforms/Scalar/InstructionCombining.cpp @@ -343,17 +343,31 @@ Instruction *InstCombiner::visitXor(BinaryOperator *I) { return Changed ? I : 0; } +// isTrueWhenEqual - Return true if the specified setcondinst instruction is +// true when both operands are equal... +// +static bool isTrueWhenEqual(Instruction *I) { + return I->getOpcode() == Instruction::SetEQ || + I->getOpcode() == Instruction::SetGE || + I->getOpcode() == Instruction::SetLE; +} + Instruction *InstCombiner::visitSetCondInst(BinaryOperator *I) { if (I->use_empty()) return 0; // Don't fix dead instructions... bool Changed = SimplifyBinOp(I); // setcc X, X if (I->getOperand(0) == I->getOperand(1)) { - bool NewVal = I->getOpcode() == Instruction::SetEQ || - I->getOpcode() == Instruction::SetGE || - I->getOpcode() == Instruction::SetLE; AddUsesToWorkList(I); // Add all modified instrs to worklist - I->replaceAllUsesWith(ConstantBool::get(NewVal)); + I->replaceAllUsesWith(ConstantBool::get(isTrueWhenEqual(I))); + return I; + } + + // setcc <global*>, 0 - Global value addresses are never null! + if (isa<GlobalValue>(I->getOperand(0)) && + isa<ConstantPointerNull>(I->getOperand(1))) { + AddUsesToWorkList(I); // Add all modified instrs to worklist + I->replaceAllUsesWith(ConstantBool::get(!isTrueWhenEqual(I))); return I; } |