[llvm-commits] [llvm] r127063 - in /llvm/trunk: lib/Transforms/InstCombine/InstCombineCompares.cpp test/Transforms/InstCombine/icmp.ll

[Duncan Sands]

Hi Nick,

> +    BinaryOperator *SRem = NULL;
> +    // icmp Y, (srem X, Y)
> +    if (BO0&&  BO0->getOpcode() == Instruction::SRem&&
> +        Op1 == BO0->getOperand(1))
> +      SRem = BO0;
> +    // icmp (srem X, Y), Y
> +    else if (BO1&&  BO1->getOpcode() == Instruction::SRem&&
> +             Op0 == BO1->getOperand(1))
> +      SRem = BO1;

the two comments seem to be the wrong way round.

Ciao, Duncan.