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

Reply via email to