about summary refs log tree commit diff
path: root/sysdeps/x86_64/fpu/s_sincosf.S
diff options
context:
space:
mode:
authorH.J. Lu <hjl.tools@gmail.com>2018-01-08 08:04:26 -0800
committerH.J. Lu <hjl.tools@gmail.com>2018-01-08 08:04:40 -0800
commitc70e4e9c9efff9df4c847dd7cfd81bae674219ab (patch)
tree46cbbfb74a8c03e933fc4245c66559def374b1a8 /sysdeps/x86_64/fpu/s_sincosf.S
parent579396ee082565ab5f42ff166a264891223b7b82 (diff)
downloadglibc-c70e4e9c9efff9df4c847dd7cfd81bae674219ab.tar.gz
glibc-c70e4e9c9efff9df4c847dd7cfd81bae674219ab.tar.xz
glibc-c70e4e9c9efff9df4c847dd7cfd81bae674219ab.zip
x86-64: Add sincosf with vector FMA
Since the x86-64 assembly version of sincosf is higly optimized with
vector instructions, there isn't much room for improvement.  However
s_sincosf.c written in C with vector math and intrinsics can be
optimized by GCC with FMA.

On Skylake, bench-sincosf reports performance improvement:

           Assembly       FMA         improvement
max        104.042       101.008         3%
min        9.426         8.586           10%
mean       20.6209       18.2238         13%

	* sysdeps/x86_64/fpu/multiarch/Makefile (libm-sysdep_routines):
	Add s_sincosf-sse2 and s_sincosf-fma.
	(CFLAGS-s_sincosf-fma.c): New.
	* sysdeps/x86_64/fpu/multiarch/s_sincosf-fma.c: New file.
	* sysdeps/x86_64/fpu/multiarch/s_sincosf-sse2.S: Likewise.
	* sysdeps/x86_64/fpu/multiarch/s_sincosf.c: Likewise.
	* sysdeps/x86_64/fpu/s_sincosf.S: Don't add alias if
	__sincosf is defined.
Diffstat (limited to 'sysdeps/x86_64/fpu/s_sincosf.S')
-rw-r--r--sysdeps/x86_64/fpu/s_sincosf.S2
1 files changed, 2 insertions, 0 deletions
diff --git a/sysdeps/x86_64/fpu/s_sincosf.S b/sysdeps/x86_64/fpu/s_sincosf.S
index f00cd0eb3c..2086e8ca5c 100644
--- a/sysdeps/x86_64/fpu/s_sincosf.S
+++ b/sysdeps/x86_64/fpu/s_sincosf.S
@@ -561,4 +561,6 @@ L(SP_ONE):
 	.type L(SP_ONE), @object
 	ASM_SIZE_DIRECTIVE(L(SP_ONE))
 
+#ifndef __sincosf
 libm_alias_float (__sincos, sincos)
+#endif