[PATCH] Remove duplicate inline implementation of issignalingf
Joseph Myers
joseph@codesourcery.com
Thu Nov 21 20:59:00 GMT 2019
On Thu, 21 Nov 2019, Paul A. Clarke wrote:
> From: "Paul A. Clarke" <pc@us.ibm.com>
>
> Very recent commit 854e91bf6b4221f424ffa13b9ef50f35623b7b74 enabled
> inline of issignalingf() in general (__issignalingf in include/math.h).
> There is another implementation for an inline use of issignalingf
> (issignalingf_inline in sysdeps/ieee754/flt-32/math_config.h)
> which could instead make use of the new enablement.
>
> Replace the use of issignalingf_inline with __issignaling.
I think it's better to replace it with calls to issignaling (the
type-generic macro), rather than to __issignalingf (the inline function).
That way, if in future GCC gets __builtin_issignaling and glibc's <math.h>
starts to use it, this code will automatically start using
__builtin_issignaling via the macro definition.
--
Joseph S. Myers
joseph@codesourcery.com
More information about the Libc-alpha
mailing list