On Mon, Jul 16, 2012 at 4:32 PM, H.J. Lu <hjl.to...@gmail.com> wrote:
> On Mon, Jul 16, 2012 at 7:14 AM, Kirill Yukhin <kirill.yuk...@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.yuk...@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.
