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.