Fix a minor bug where we would go into infinite loops on some constants

llvm-svn: 14638
This commit is contained in:
Chris Lattner 2004-07-06 07:11:42 +00:00
parent 7fd5f0745a
commit 98c6bdf251
1 changed files with 2 additions and 1 deletions

View File

@ -846,7 +846,8 @@ Instruction *InstCombiner::visitDiv(BinaryOperator &I) {
Instruction *InstCombiner::visitRem(BinaryOperator &I) {
if (I.getType()->isSigned())
if (Value *RHSNeg = dyn_castNegVal(I.getOperand(1)))
if (RHSNeg != I.getOperand(1)) { // Avoid problems with MININT
if (!isa<ConstantSInt>(RHSNeg) ||
cast<ConstantSInt>(RHSNeg)->getValue() >= 0) {
// X % -Y -> X % Y
AddUsesToWorkList(I);
I.setOperand(1, RHSNeg);