[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

[Evan Cheng]

On Jul 28, 2009, at 10:20 PM, John McCall wrote:

> Eric Christopher wrote:
>> _foo:
>> 	ptest 	%xmm1, %xmm0
>> 	sete	%al
>> 	movzbl	%al, %eax
>> 	testl	%eax, %eax
>> 	sete	%al
>> 	movzbl	%al, %eax
>> 	leal	4(,%eax,4), %eax
>> 	ret
>>
>> Honestly it's the movzbl instructions that irritate me the most, but
>> there's no setcc that will set an entire register :(
>>
>
> I'm just a naive front-end guy, but shouldn't we be optimizing
>    movzbl %al, %eax
>    testl %eax, %eax
> into
>    testb %al, %al
> ?

Yes, we should fix it. Can one of you file a bug?

Thanks,

Evan

>
> John.
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits