diff options
| author | Joseph Myers <joseph@codesourcery.com> | 2020-07-06 16:18:02 +0000 |
|---|---|---|
| committer | Joseph Myers <joseph@codesourcery.com> | 2020-07-06 16:18:02 +0000 |
| commit | 6c010c5dde1735f93cc3a6597cdcc2b482af85f8 (patch) | |
| tree | 5f0a95e22d85c21352efccfde78f2f515cdba433 /sysdeps/ieee754/ldbl-128ibm/s_getpayloadl.c | |
| parent | 28c13ae5bbc81aa2ae67890ce53e65257d4703a4 (diff) | |
| download | glibc-6c010c5dde1735f93cc3a6597cdcc2b482af85f8.zip glibc-6c010c5dde1735f93cc3a6597cdcc2b482af85f8.tar.gz glibc-6c010c5dde1735f93cc3a6597cdcc2b482af85f8.tar.bz2 | |
Use C2x return value from getpayload of non-NaN (bug 26073).
In TS 18661-1, getpayload had an unspecified return value for a
non-NaN argument, while C2x requires the return value -1 in that case.
This patch implements the return value of -1. I don't think this is
worth having a new symbol version that's an alias of the old one,
although occasionally we do that in such cases where the new function
semantics are a refinement of the old ones (to avoid programs relying
on the new semantics running on older glibc versions but not behaving
as intended).
Tested for x86_64 and x86; also ran math/ tests for aarch64 and
powerpc.
Diffstat (limited to 'sysdeps/ieee754/ldbl-128ibm/s_getpayloadl.c')
| -rw-r--r-- | sysdeps/ieee754/ldbl-128ibm/s_getpayloadl.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/sysdeps/ieee754/ldbl-128ibm/s_getpayloadl.c b/sysdeps/ieee754/ldbl-128ibm/s_getpayloadl.c index 3b5a1a8..abbd694 100644 --- a/sysdeps/ieee754/ldbl-128ibm/s_getpayloadl.c +++ b/sysdeps/ieee754/ldbl-128ibm/s_getpayloadl.c @@ -27,6 +27,9 @@ __getpayloadl (const long double *x) double xhi = ldbl_high (*x); uint64_t ix; EXTRACT_WORDS64 (ix, xhi); + if ((ix & 0x7ff0000000000000ULL) != 0x7ff0000000000000ULL + || (ix & 0xfffffffffffffULL) == 0) + return -1; ix &= 0x7ffffffffffffULL; if (FIX_INT_FP_CONVERT_ZERO && ix == 0) return 0.0L; |