[llvm-commits] [llvm] r77407 - in /llvm/trunk: include/llvm/IntrinsicsX86.td lib/Target/X86/X86ISelLowering.cpp lib/Target/X86/X86ISelLowering.h lib/Target/X86/X86InstrSSE.td test/CodeGen/X86/sse41.ll

[Chris Lattner]

On Jul 28, 2009, at 10:58 PM, Evan Cheng wrote:

>>
>> Two possible approaches:
>>
>> 1. The intrinsic could be declared to return i8 instead of i32, that
>> would be a hack around the problem, but would work be reasonable
>> since the X86 setcc instructions set an i8.
>>
>> 2. We could lower the ptestz intrinisic at sdisel time instead of in
>> legalize like we do for call and arguments.
>>
>> I don't see a reason not to do #2: it seems like a general win for
>> intrinsics that are lowered like this.  What do you guys think?
>
> #2 makes sense to me.

Can you file a bug about it? ;-)

-Chris