This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, libcpp]: Use x86 __builtin_ia32_pcmpestri128 instead of asm.
- From: Richard Henderson <rth at redhat dot com>
- To: Uros Bizjak <ubizjak at gmail dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Mon, 18 Jun 2012 15:07:35 -0700
- Subject: Re: [PATCH, libcpp]: Use x86 __builtin_ia32_pcmpestri128 instead of asm.
- References: <CAFULd4a=HVQrmm9wEsoSgPEeVB6x9U+yJk1OjUOsTSeiiquJQw@mail.gmail.com>
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_loaddqu ((const char *) s);
> + index = __builtin_ia32_pcmpestri128 (search, 4, sv, 16, 0);
> +
Surely the comment can be removed too then?
r~