aboutsummaryrefslogtreecommitdiff
path: root/sysdeps/i386/i686/fpu/multiarch
diff options
context:
space:
mode:
authorPaul Zimmermann <Paul.Zimmermann@inria.fr>2024-10-08 11:28:04 +0200
committerPaul Zimmermann <Paul.Zimmermann@inria.fr>2024-10-11 11:12:32 +0200
commit392b3f0971764a0dff22942a0d9726a7d5cb4e0f (patch)
treeda77493f40d6c4b7d93a1657acbc20882c63e80d /sysdeps/i386/i686/fpu/multiarch
parente850abd8d897279b7aa21d25fb2bbf13b9d9b71c (diff)
downloadglibc-392b3f0971764a0dff22942a0d9726a7d5cb4e0f.zip
glibc-392b3f0971764a0dff22942a0d9726a7d5cb4e0f.tar.gz
glibc-392b3f0971764a0dff22942a0d9726a7d5cb4e0f.tar.bz2
replace tgammaf by the CORE-MATH implementation
The CORE-MATH implementation is correctly rounded (for any rounding mode). This can be checked by exhaustive tests in a few minutes since there are less than 2^32 values to check against for example GNU MPFR. This patch also adds some bench values for tgammaf. Tested on x86_64 and x86 (cfarm26). With the initial GNU libc code it gave on an Intel(R) Core(TM) i7-8700: "tgammaf": { "": { "duration": 3.50188e+09, "iterations": 2e+07, "max": 602.891, "min": 65.1415, "mean": 175.094 } } With the new code: "tgammaf": { "": { "duration": 3.30825e+09, "iterations": 5e+07, "max": 211.592, "min": 32.0325, "mean": 66.1649 } } With the initial GNU libc code it gave on cfarm26 (i686): "tgammaf": { "": { "duration": 3.70505e+09, "iterations": 6e+06, "max": 2420.23, "min": 243.154, "mean": 617.509 } } With the new code: "tgammaf": { "": { "duration": 3.24497e+09, "iterations": 1.8e+07, "max": 1238.15, "min": 101.155, "mean": 180.276 } } Signed-off-by: Alexei Sibidanov <sibid@uvic.ca> Signed-off-by: Paul Zimmermann <Paul.Zimmermann@inria.fr> Changes in v2: - include <math.h> (fix the linknamespace failures) - restored original benchtests/strcoll-inputs/filelist#en_US.UTF-8 file - restored original wrapper code (math/w_tgammaf_compat.c), except for the dealing with the sign - removed the tgammaf/float entries in all libm-test-ulps files - address other comments from Joseph Myers (https://sourceware.org/pipermail/libc-alpha/2024-July/158736.html) Changes in v3: - pass NULL argument for signgam from w_tgammaf_compat.c - use of math_narrow_eval - added more comments Changes in v4: - initialize local_signgam to 0 in math/w_tgamma_template.c - replace sysdeps/ieee754/dbl-64/gamma_productf.c by dummy file Changes in v5: - do not mention local_signgam any more in math/w_tgammaf_compat.c - initialize local_signgam to 1 instead of 0 in w_tgamma_template.c and added comment Changes in v6: - pass NULL as 2nd argument of __ieee754_gammaf_r in w_tgammaf_compat.c, and check for NULL in e_gammaf_r.c Changes in v7: - added Signed-off-by line for Alexei Sibidanov (author of the code) Changes in v8: - added Signed-off-by line for Paul Zimmermann (submitted of the patch) Changes in v9: - address comments from review by Adhemerval Zanella Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
Diffstat (limited to 'sysdeps/i386/i686/fpu/multiarch')
-rw-r--r--sysdeps/i386/i686/fpu/multiarch/libm-test-ulps4
1 files changed, 0 insertions, 4 deletions
diff --git a/sysdeps/i386/i686/fpu/multiarch/libm-test-ulps b/sysdeps/i386/i686/fpu/multiarch/libm-test-ulps
index 5c41bf3..0a813aa 100644
--- a/sysdeps/i386/i686/fpu/multiarch/libm-test-ulps
+++ b/sysdeps/i386/i686/fpu/multiarch/libm-test-ulps
@@ -1701,25 +1701,21 @@ ldouble: 4
Function: "tgamma":
double: 9
-float: 8
float128: 4
ldouble: 5
Function: "tgamma_downward":
double: 9
-float: 7
float128: 5
ldouble: 6
Function: "tgamma_towardzero":
double: 9
-float: 7
float128: 5
ldouble: 6
Function: "tgamma_upward":
double: 8
-float: 8
float128: 4
ldouble: 5