Re: [PATCH, libcpp]: Use x86 __builtin_ia32_pcmpestri128 instead of asm.

2012-06-19 Thread Richard Henderson
On 2012-06-18 23:38, Uros Bizjak wrote: > On Tue, Jun 19, 2012 at 12:07 AM, Richard Henderson wrote: >> On 2012-06-18 13:19, Uros Bizjak wrote: >>>/* ??? The builtin doesn't understand that the PCMPESTRI read from >>>memory need not be aligned. */ >>> - __asm ("%vpcmpestri $0

Re: [PATCH, libcpp]: Use x86 __builtin_ia32_pcmpestri128 instead of asm.

2012-06-19 Thread Uros Bizjak
On Tue, Jun 19, 2012 at 8:38 AM, Uros Bizjak wrote: > On Tue, Jun 19, 2012 at 12:07 AM, Richard Henderson wrote: >> On 2012-06-18 13:19, Uros Bizjak wrote: >>>        /* ??? The builtin doesn't understand that the PCMPESTRI read from >>>        memory need not be aligned.  */ >>> -      __asm ("%

Re: [PATCH, libcpp]: Use x86 __builtin_ia32_pcmpestri128 instead of asm.

2012-06-18 Thread Uros Bizjak
On Tue, Jun 19, 2012 at 12:07 AM, Richard Henderson wrote: > On 2012-06-18 13:19, Uros Bizjak wrote: >>        /* ??? The builtin doesn't understand that the PCMPESTRI read from >>        memory need not be aligned.  */ >> -      __asm ("%vpcmpestri $0, (%1), %2" >> -          : "=c"(index) : "r"(

Re: [PATCH, libcpp]: Use x86 __builtin_ia32_pcmpestri128 instead of asm.

2012-06-18 Thread Richard Henderson
On 2012-06-18 13:19, Uros Bizjak wrote: >/* ??? The builtin doesn't understand that the PCMPESTRI read from >memory need not be aligned. */ > - __asm ("%vpcmpestri $0, (%1), %2" > - : "=c"(index) : "r"(s), "x"(search), "a"(4), "d"(16)); > + sv = __builtin_ia32_lo