diff options
author | Joseph Myers <joseph@codesourcery.com> | 2021-09-10 20:56:22 +0000 |
---|---|---|
committer | Joseph Myers <joseph@codesourcery.com> | 2021-09-10 20:56:22 +0000 |
commit | abd383584b16dd0fb1bbf40e4ece65ebe7b839ec (patch) | |
tree | 01a62d12933145f8c996bc2025a1646e98622fbd /sysdeps/i386 | |
parent | c9fef4b7d1d0f2dad192c74f06102752247677a9 (diff) | |
download | glibc-abd383584b16dd0fb1bbf40e4ece65ebe7b839ec.tar.gz glibc-abd383584b16dd0fb1bbf40e4ece65ebe7b839ec.tar.xz glibc-abd383584b16dd0fb1bbf40e4ece65ebe7b839ec.zip |
Add narrowing square root functions
This patch adds the narrowing square root functions from TS 18661-1 / TS 18661-3 / C2X to glibc's libm: fsqrt, fsqrtl, dsqrtl, f32sqrtf64, f32sqrtf32x, f32xsqrtf64 for all configurations; f32sqrtf64x, f32sqrtf128, f64sqrtf64x, f64sqrtf128, f32xsqrtf64x, f32xsqrtf128, f64xsqrtf128 for configurations with _Float64x and _Float128; __f32sqrtieee128 and __f64sqrtieee128 aliases in the powerpc64le case (for calls to fsqrtl and dsqrtl when long double is IEEE binary128). Corresponding tgmath.h macro support is also added. The changes are mostly similar to those for the other narrowing functions previously added, so the description of those generally applies to this patch as well. However, the not-actually-narrowing cases (where the two types involved in the function have the same floating-point format) are aliased to sqrt, sqrtl or sqrtf128 rather than needing a separately built not-actually-narrowing function such as was needed for add / sub / mul / div. Thus, there is no __nldbl_dsqrtl name for ldbl-opt because no such name was needed (whereas the other functions needed such a name since the only other name for that entry point was e.g. f32xaddf64, not reserved by TS 18661-1); the headers are made to arrange for sqrt to be called in that case instead. The DIAG_* calls in sysdeps/ieee754/soft-fp/s_dsqrtl.c are because they were observed to be needed in GCC 7 testing of riscv32-linux-gnu-rv32imac-ilp32. The other sysdeps/ieee754/soft-fp/ files added didn't need such DIAG_* in any configuration I tested with build-many-glibcs.py, but if they do turn out to be needed in more files with some other configuration / GCC version, they can always be added there. I reused the same test inputs in auto-libm-test-in as for non-narrowing sqrt rather than adding extra or separate inputs for narrowing sqrt. The tests in libm-test-narrow-sqrt.inc also follow those for non-narrowing sqrt. Tested as followed: natively with the full glibc testsuite for x86_64 (GCC 11, 7, 6) and x86 (GCC 11); with build-many-glibcs.py with GCC 11, 7 and 6; cross testing of math/ tests for powerpc64le, powerpc32 hard float, mips64 (all three ABIs, both hard and soft float). The different GCC versions are to cover the different cases in tgmath.h and tgmath.h tests properly (GCC 6 has _Float* only as typedefs in glibc headers, GCC 7 has proper _Float* support, GCC 8 adds __builtin_tgmath).
Diffstat (limited to 'sysdeps/i386')
-rw-r--r-- | sysdeps/i386/fpu/w_sqrt.c | 2 | ||||
-rw-r--r-- | sysdeps/i386/fpu/w_sqrt_compat.c | 2 |
2 files changed, 4 insertions, 0 deletions
diff --git a/sysdeps/i386/fpu/w_sqrt.c b/sysdeps/i386/fpu/w_sqrt.c index 8bef04e68a..644c7c870c 100644 --- a/sysdeps/i386/fpu/w_sqrt.c +++ b/sysdeps/i386/fpu/w_sqrt.c @@ -2,7 +2,9 @@ most internal uses in glibc, but not for sqrt itself. */ #define NO_MATH_REDIRECT #define __ieee754_sqrt __avoid_ieee754_sqrt +#define f32xsqrtf64 __hide_f32xsqrtf64 #include <math.h> +#undef f32xsqrtf64 #include <math_private.h> #undef __ieee754_sqrt extern double __ieee754_sqrt (double); diff --git a/sysdeps/i386/fpu/w_sqrt_compat.c b/sysdeps/i386/fpu/w_sqrt_compat.c index dd485f4b88..f5d3cf91d1 100644 --- a/sysdeps/i386/fpu/w_sqrt_compat.c +++ b/sysdeps/i386/fpu/w_sqrt_compat.c @@ -2,7 +2,9 @@ most internal uses in glibc, but not for sqrt itself. */ #define NO_MATH_REDIRECT #define __ieee754_sqrt __avoid_ieee754_sqrt +#define f32xsqrtf64 __hide_f32xsqrtf64 #include <math.h> +#undef f32xsqrtf64 #include <math_private.h> #undef __ieee754_sqrt extern double __ieee754_sqrt (double); |