Hi!

On Wed, Jan 12, 2022 at 10:02:36AM +0100, Martin Liška wrote:
> -      error ("%qs requires ISA 3.0 IEEE 128-bit floating point", name);
> +      error ("%qs requires ISA 3.0 IEEE 128-bit floating-point", name);

This change is incorrect.  Floating point is not an adjective here.

It is probably best to just rewrite this text, of course.  Maybe
"%qs requires quad-precision floating-point arithmetic" or something
like that (that it was first introduced in Power ISA 3.0 is dated
already, for example).

> -      error ("%<%s%> not supported with %<-msoft-float%>",
> +      error ("%qs not supported with %<-msoft-float%>",

This change is trivial and obviously correct.  Please just commit such
changes, *not* mixed in with anything else though.  (And if you aren't
sure something is trivial or obviously correct, it is not!)

> -  return scalar_extract_exp (source);        /* { dg-error "requires ISA 3.0 
> IEEE 
> 128-bit floating point" } */
> +  return scalar_extract_exp (source);        /* { dg-error "requires ISA 3.0 
> IEEE 
> 128-bit floating-point" } */

Your patch is malformed.  Please fix your mailer.


Segher

Reply via email to