On 9/29/17 5:31 PM, Segher Boessenkool wrote: >> + /* PowerPC 64-bit LE requires at least ISA 2.07. */ >> + const char *default_cpu = ((!TARGET_POWERPC64) >> + ? "powerpc" >> + : ((BYTES_BIG_ENDIAN) >> + ? "powerpc64" >> + : "powerpc64le")); > > Please remove the parens around !TARGET_POWERPC64 and BYTES_BIG_ENDIAN > while you're at it (one of the copies you removed had that already :-) )
Done. > Looks great to me, please commit. But hold off until Monday please, it > will interfere with testing otherwise. Ok, committed now (Monday). I'd also like to back port this to the GCC 7 and 6 release branches, where the earlier fix was also back ported to. Ok there after a week or so of burn in on trunk? Thanks. Peter