[PATCH] rs6000: Builtins test changes for BFP scalar tests

Segher Boessenkool segher@kernel.crashing.org
Thu Nov 18 21:18:39 GMT 2021


On Thu, Nov 18, 2021 at 07:32:27AM -0600, Bill Schmidt wrote:
> >>>   error: '__builtin_vsx_scalar_extract_sig' requires the '-mcpu=power9' option and either the '-m64' or '-mpowerpc64' option
> >>>   note: builtin '__builtin_vec_scalar_extract_sig' requires builtin '__builtin_vsx_scalar_extract_sig'
> >> The rhs in the note does not *exist*, as far as the user is concerned.
> >> One builtin requiring another is all gobbledygook.
> > As stated above, this isn't something new that I've added.  That's what
> > we already do.  It's how the overload error messages have always been.
> 
> That said, I don't like the "requires" language at all, either.  How about
> replacing "builtin X requires builtin Y" with "overloaded builtin X is
> implemented by builtin Y" as a better explanation?

That is better (although builtin Y *does not exist* as far as the user
is concerned: it is not documented, and you cannot write it in source
code afaics).


Segher


More information about the Gcc-patches mailing list