Re: PATCH: Add -mavx2 and properly check numbers of mask bits

2011-08-08 Thread Andreas Schwab
"H.J. Lu" writes: > #define OPTION_MASK_ISA_XOP (1 << 31) 1 << 31 is undefined. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "And now for something completely different."

Re: PATCH: Add -mavx2 and properly check numbers of mask bits

2011-08-07 Thread Hans-Peter Nilsson
On Sun, 7 Aug 2011, H.J. Lu wrote: > Hi, > > opth-gen.awk has > > print "#define " mask name " (1 << " masknum[vname]++ ")" > > and int has 32bits. We should check > > if (masknum[var] > 32) > > instead of > > if (masknum[var] > 31) IIUC the (int32_t) sign-bit is supposed to be reserved and this i

PATCH: Add -mavx2 and properly check numbers of mask bits

2011-08-07 Thread H.J. Lu
Hi, opth-gen.awk has print "#define " mask name " (1 << " masknum[vname]++ ")" and int has 32bits. We should check if (masknum[var] > 32) instead of if (masknum[var] > 31) Now, I got #define OPTION_MASK_ISA_X32 (1 << 30) #define OPTION_MASK_ISA_XOP (1 << 31) in options.h. OK for trunk? T