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
jos...@codesourcery.com

Reply via email to