On Mon, Oct 02, 2017 at 12:00:55PM -0500, Peter Bergner wrote: > 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?
Certainly. Thanks! Segher