On Sat, Sep 17, 2011 at 11:26 PM, H.J. Lu <hjl.to...@gmail.com> wrote:

>>> Agreed. Some parets are missing, though:
>>>
>>> -  unsigned long long tmp = (__X) ^ (__X - 1);
>>> -  return tmp;
>>> +  unsigned long long __tmp = (__X) ^ (__X - 1);
>>> +  return __tmp;
>>
>> There is none missing.  This is not a macro.
>>
>
> Here is the updated patch.  Tested on Linux/x86-64.  OK
> for trunk?

> 2011-09-17  H.J. Lu  <hongjiu...@intel.com>
>
>        * config/i386/bmiintrin.h: Remove tmp.
>        * config/i386/tbmintrin.h: Likewise.

OK.

Thanks,
Uros.

Reply via email to