[PATCH, version 2], Add support for _Float<N> and _Float<N>X sqrt, fma, fmin, fmax built-in functions

Joseph Myers joseph@codesourcery.com
Thu Oct 19 23:00:00 GMT 2017


On Thu, 19 Oct 2017, Michael Meissner wrote:

>     1)	I switched to use DEF_EXT_LIB_BUILTIN to declare the _Float<N> and
> 	_Float<N>X functions.  This allows treating __builtin_sqrtf128 the same
> 	as sqrtf128.

It's not correct to do that unconditionally for all the existing 
DEF_GCC_FLOATN_NX_BUILTINS functions.  There should not be a public 
huge_valf128 function any more than a public huge_val function, just 
__builtin_huge_valf128.

Rather, you should add a new DEF_EXT_LIB_FLOATN_NX_BUILTINS.  It should be 
used by the new functions, and by the existing copysign / fabs / nan 
functions.  It should not be used by the existing huge_val / inf / nans 
functions.

-- 
Joseph S. Myers
joseph@codesourcery.com



More information about the Gcc-patches mailing list