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


Reply via email to