diff options
author | Joseph Myers <joseph@codesourcery.com> | 2017-10-11 17:46:19 +0000 |
---|---|---|
committer | Joseph Myers <joseph@codesourcery.com> | 2017-10-11 17:46:19 +0000 |
commit | bc9620d040b7494f457ccb750c9797b47ed76ada (patch) | |
tree | b2a800a1b6bc16ee769881d38b4c160a4bd8b92b /math/libm-test-support.c | |
parent | 09c76a74099826f4c6e1c4c431d7659f78112862 (diff) | |
download | glibc-bc9620d040b7494f457ccb750c9797b47ed76ada.tar.gz glibc-bc9620d040b7494f457ccb750c9797b47ed76ada.tar.xz glibc-bc9620d040b7494f457ccb750c9797b47ed76ada.zip |
Run libm long double tests for long double = double.
All interfaces in the glibc API ought to be covered by the testsuite, even where they alias other interfaces. This patch arranges for libm tests to be run for long double when it aliases double; previously those tests were run only for the floating-point types with distinct formats. The long double tests are made to use the double ulps values in this case, as having a separate duplicate set of ulps for them seems unnecessary; to accommodate that, the test-<type>.h headers now specify the macro indexing into the ulps array explicitly instead of having it computed from PREFIX. Nothing special is done about vector function tests. None are supported for any long double = double platforms, and supporting vector functions for a type alias such as _Float32 would not simply fall out of adding the scalar aliases for that type - it would require vector function wrappers like those for *_finite (or, better, a new GCC feature to allow specifying the asm name for vector functions independently of that for scalar ones, as previously discussed), so it seems reasonable to require the sysdeps makefile setting of libmvec-tests to be updated if any such tests are to be run for type aliases. Tested for x86_64 and arm. * math/Makefile (test-types-basic): New variable. (test-types): Likewise. (libm-test-support): Use $(test-types) instead of $(types). (libm-tests-base-normal): Likewise. (libm-tests-base-finite): Likewise. (libm-tests-base-inline): Likewise. (generated): Likewise. ($(objpfx)libm-test-support-$(t).c): Likewise. (libm-tests-for-type iterator): Likewise. (libm-test-support iterator): Likewise. * math/libm-test-support.c (ulp_i_idx): Use ULP_I_IDX. (ulp_idx): Use ULP_IDX. * math/test-ldouble.h: Include <float.h>. (TYPE_STR): Define conditional on [LDBL_MANT_DIG == DBL_MANT_DIG]. (ULP_IDX): New macro. (ULP_I_IDX): Likewise. * math/test-double.h (ULP_IDX): Likewise. (ULP_I_IDX): Likewise. * math/test-float.h (ULP_IDX): Likewise. (ULP_I_IDX): Likewise. * math/test-float128.h (ULP_IDX): Likewise. (ULP_I_IDX): Likewise.
Diffstat (limited to 'math/libm-test-support.c')
-rw-r--r-- | math/libm-test-support.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/math/libm-test-support.c b/math/libm-test-support.c index 9ce3eb1c04..ede0d161a2 100644 --- a/math/libm-test-support.c +++ b/math/libm-test-support.c @@ -45,6 +45,9 @@ TYPE_STR: The name of the type as used in ulps files, as a string. + ULP_IDX, ULP_I_IDX: The array indexes for ulps values for this + function. + LIT: Append the correct suffix to a literal. LITM: Append the correct suffix to an M_* macro name. @@ -191,8 +194,8 @@ compare_ulp_data (const void *key, const void *ulp) return strcmp (keystr, ulpdat->name); } -static const int ulp_i_idx = __CONCATX (ULP_I_, PREFIX); -static const int ulp_idx = __CONCATX (ULP_, PREFIX); +static const int ulp_i_idx = ULP_I_IDX; +static const int ulp_idx = ULP_IDX; /* Return the ulps for NAME in array DATA with NMEMB elements, or 0 if no ulps listed. */ |