[PATCH, i386, PR53877] New intrinsics for LZCNT
Uros Bizjak
ubizjak@gmail.com
Mon Jul 16 14:53:00 GMT 2012
On Mon, Jul 16, 2012 at 4:32 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Mon, Jul 16, 2012 at 7:14 AM, Kirill Yukhin <kirill.yukhin@gmail.com> wrote:
>> Hello guys,
>>
>> Here is a tiny patch which adds two new intrinsics which were
>> introduced in recent spec [1].
>> They're aliased to the existing __lzcnt_* and live under same CPUID.
>>
>> ChangeLog entry is:
>> 2012-07-16 Kirill Yukhin <kirill.yukhin@intel.com>
>>
>> PR target/53877
>> * config/i386/lzcntintrin.h (_lzcnt_u32): New.
>> (_lzcnt_u64): Ditto.
>>
>> Is it OK for trunk?
>>
>
> I think we should add a testcase for each. Also it should
> also be applied on 4.7.
This patch just calls __builtin_clz{,ll}, for which we already have
test coverage. Also, please note a couple of sse-*.c testcases that
check compilation of all intrinsics in one shot.
So, OK for mainline and 4.7.
Thanks,
Uros.
More information about the Gcc-patches
mailing list