Don't apply this transform to vectors. Fixes PR2756.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@55690 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Nick Lewycky 2008-09-03 06:24:21 +00:00
parent 83c05e5d30
commit 23c04309fb

View File

@ -2948,8 +2948,9 @@ Instruction *InstCombiner::visitSRem(BinaryOperator &I) {
return common; return common;
if (Value *RHSNeg = dyn_castNegVal(Op1)) if (Value *RHSNeg = dyn_castNegVal(Op1))
if (!isa<ConstantInt>(RHSNeg) || if (!isa<Constant>(RHSNeg) ||
cast<ConstantInt>(RHSNeg)->getValue().isStrictlyPositive()) { (isa<ConstantInt>(RHSNeg) &&
cast<ConstantInt>(RHSNeg)->getValue().isStrictlyPositive())) {
// X % -Y -> X % Y // X % -Y -> X % Y
AddUsesToWorkList(I); AddUsesToWorkList(I);
I.setOperand(1, RHSNeg); I.setOperand(1, RHSNeg);
@ -5495,7 +5496,7 @@ Instruction *InstCombiner::visitICmpInst(ICmpInst &I) {
if (Op0I->getOpcode() == Op1I->getOpcode() && Op0I->hasOneUse() && if (Op0I->getOpcode() == Op1I->getOpcode() && Op0I->hasOneUse() &&
Op1I->hasOneUse() && Op0I->getOperand(1) == Op1I->getOperand(1) && Op1I->hasOneUse() && Op0I->getOperand(1) == Op1I->getOperand(1) &&
I.isEquality()) { I.isEquality()) {
switch (Op0I->getOpcode()) { switch (Op0I->getOpcode()) {
default: break; default: break;
case Instruction::Add: case Instruction::Add:
case Instruction::Sub: case Instruction::Sub: