[llvm-commits] [llvm] r58351 - in /llvm/trunk: lib/Transforms/Scalar/InstructionCombining.cpp test/Transforms/InstCombine/logical-select.ll

[Dan Gohman]

On Oct 28, 2008, at 10:30 PM, Duncan Sands wrote:

> Hi Dan,
>
>> +    // (A & sext(C0)) | (B & ~sext(C0) ->  C0 ? A : B
>
> missing closing parenthesis after ~sext(C0).

Fixed, thanks.

>  Also, why
> use C0 and not C?

It's meant to be operand 0 of C, where C is from
this comment:
   // (A & C)|(B & D)
on the if statement that encloses this code.

Dan