[llvm-commits] [llvm] r75531 - in /llvm/trunk: lib/Transforms/Scalar/InstructionCombining.cpp test/Transforms/InstCombine/zext-bool-add-sub.ll

[Chris Lattner]

On Jul 13, 2009, at 3:50 PM, Eli Friedman wrote:

> On Mon, Jul 13, 2009 at 3:35 PM, Chris Lattner<clattner at apple.com>  
> wrote:
>> I think "select c, -1, 0" -> "sext c" and  "select c, 0, -1" ->  
>> "sext !
>> c" would make sense, what do you think?
>
> We intentionally do the opposite at the moment.

That probably dates back to when 'sext from bool' was new.  Pre-2.0 we  
had no such operation, and there were bugs immediately after the  
conversion.

-Chris