about summary refs log tree commit diff
path: root/ChangeLog
diff options
context:
space:
mode:
authorWilco Dijkstra <wdijkstr@arm.com>2018-08-15 16:01:21 +0100
committerWilco Dijkstra <wdijkstr@arm.com>2018-08-15 16:01:21 +0100
commit126c4e3f804881f9fbc4eb71787f41793d2d7be5 (patch)
treea8925aeea4de10fc9fdf788cabc89118e06cd892 /ChangeLog
parent49acec179ca9fb2da961b78f56ef5ce501bbb50b (diff)
downloadglibc-126c4e3f804881f9fbc4eb71787f41793d2d7be5.tar.gz
glibc-126c4e3f804881f9fbc4eb71787f41793d2d7be5.tar.xz
glibc-126c4e3f804881f9fbc4eb71787f41793d2d7be5.zip
Use generic sinf/cosf in lgammaf_r
The internal functions __kernel_sinf and __kernel_cosf are used only by
lgammaf_r.  Removing the internal functions and using the generic sinf
and cosf is better overall.  Benchmarking on Cortex-A72 shows the generic
sinf and cosf are 1.4x and 2.3x faster in the range |x| < PI/4, and 0.66x
and 1.1x for |x| < PI/2, so it should make lgammaf_r faster on average.

GLIBC regression tests pass on AArch64.

	* sysdeps/ieee754/flt-32/e_lgammaf_r.c (sin_pif): Use __sinf/__cosf.
	* sysdeps/ieee754/flt-32/k_cosf.c (__kernel_cosf): Remove all code.
	* sysdeps/ieee754/flt-32/k_sinf.c (__kernel_sinf): Likewise.
Diffstat (limited to 'ChangeLog')
-rw-r--r--ChangeLog6
1 files changed, 6 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog
index f1cc02c1ef..21404da510 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2018-08-15  Wilco Dijkstra  <wdijkstr@arm.com>
 
+	* sysdeps/ieee754/flt-32/e_lgammaf_r.c (sin_pif): Use __sinf/__cosf.
+	* sysdeps/ieee754/flt-32/k_cosf.c (__kernel_cosf): Remove all code.
+	* sysdeps/ieee754/flt-32/k_sinf.c (__kernel_sinf): Likewise.
+
+2018-08-15  Wilco Dijkstra  <wdijkstr@arm.com>
+
 	* sysdeps/x86_64/fpu/libm-test-ulps: Regenerate to fix spaces.
 
 2018-08-14  Florian Weimer  <fweimer@redhat.com>