[COMMITTED] Use C99 math macros
Wilco Dijkstra
wdijkstr@arm.com
Wed Jun 3 15:48:00 GMT 2015
> Joseph Myers wrote:
> On Wed, 3 Jun 2015, Wilco Dijkstra wrote:
>
> > diff --git a/sysdeps/ieee754/ldbl-opt/nldbl-finite.c b/sysdeps/ieee754/ldbl-opt/nldbl-
> finite.c
>
> > - return __finite (x);
> > + return finite (x);
>
> Are you sure about this change? "finite", as opposed to isfinite, is a
> non-type-generic BSD function; I'd expect this to cause the "finite"
> function name to be used for the call instead of __finite. (Using
> isfinite here should be fine.)
Good catch - that's a typo, I meant isfinite indeed. There are a few other calls
to finite so I don't think it causes namespace issues:
sysdeps/ieee754/ldbl-128ibm/s_fmal.c: if ((finite ((double)x) && finite ((double)y)) && isinf
((double)z))
sysdeps/ieee754/ldbl-96/s_fma.c: if (finite (x) && finite (y))
I'll fix these as well so they will become inlineable.
[ I guess this file is dead: sysdeps/ieee754/support.c: if(!finite(x)) return(x); ]
Wilco
More information about the Libc-alpha
mailing list