[PATCH] Inline C99 math functions

Wilco Dijkstra wdijkstr@arm.com
Mon Jun 15 13:07:00 GMT 2015


Add inlining of the C99 math functions isinf/isnan/signbit/isfinite/isnormal/fpclassify using GCC
built-ins when available. Since going through the PLT is expensive for these small functions,
inlining results in major speedups (about 7x on Cortex-A57 for isinf). The GCC built-ins are not
correct if signalling NaN support is required, and thus are turned off in that case (see GCC bug
66462). The test-snan.c tests sNaNs and so must be explicitly built with -fsignaling-nans.

As a result of this many target overrides and the various __isnan/__finite inlines in math_private.h
are no longer required. If agreed we could remove all this code and only keep the generic definition
of isinf/etc which will use the builtin.

Tested on AArch64. OK for commit?

ChangeLog: 
2015-06-15  Wilco Dijkstra  <wdijkstr@arm.com>

	* math/Makefile: Build test-snan.c with -fsignaling-nans.	
	* math/math.h (fpclassify): Use __builtin_fpclassify when
	available.  (signbit): Use __builtin_signbit(f/l).
	(isfinite): Use__builtin_isfinite.  (isnormal): Use
	__builtin_isnormal.  (isnan): Use __builtin_isnan.
	(isinf): Use __builtin_isinf_sign.

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: Enable-C99-math-inlines.txt
URL: <http://sourceware.org/pipermail/libc-alpha/attachments/20150615/1d61c5b3/attachment.txt>


More information about the Libc-alpha mailing list