On Wed, May 29, 2019 at 10:52:38PM -0500, Bill Schmidt wrote:
>       * lib/target-supports.exp (check_powerpc_future_hw_available):
>       New.

That fits on one line just fine?

> +# Return 1 if this is a PowerPC target supporting -mfuture.
> +# Limit this to 64-bit linux systems for now until other
> +# targets support FUTURE.
> +
> +proc check_effective_target_powerpc_future_ok { } {
> +    if { ([istarget powerpc64*-*-linux*]) } {

It should be limited to powerpc64le-*-linux, or at least to ELFv2, right?

Not that that should matter at all: you pass -mfuture, and the compiler
will not allow that on unsupported configurations anyway (right? :-) )

But, okay for trunk.  Thanks!


Segher

Reply via email to