diff options
| author | Joseph Myers <joseph@codesourcery.com> | 2016-06-08 22:24:06 +0000 |
|---|---|---|
| committer | Joseph Myers <joseph@codesourcery.com> | 2016-06-08 22:24:06 +0000 |
| commit | df179d88080ad7a733edccc655cf5f6bbec6f8fc (patch) | |
| tree | d1f7b50279ad7221529019078ad2a170fcea5aaf | |
| parent | 9bd3ef8e19b19fb3cfcf99a33a1493035891b990 (diff) | |
| download | glibc-df179d88080ad7a733edccc655cf5f6bbec6f8fc.tar glibc-df179d88080ad7a733edccc655cf5f6bbec6f8fc.tar.gz glibc-df179d88080ad7a733edccc655cf5f6bbec6f8fc.tar.bz2 glibc-df179d88080ad7a733edccc655cf5f6bbec6f8fc.zip | |
Fix i386/x86_64 logl (sNaN) (bug 20227).
The i386/x86_64 versions of logl return sNaN for sNaN input. This
patch fixes them to add a NaN input to itself so that qNaN is returned
in this case.
Tested for x86_64 and x86 (including a build for i586 to cover the
non-i686 logl version).
[BZ #20227]
* sysdeps/i386/fpu/e_logl.S (__ieee754_logl): Add NaN input to
itself.
* sysdeps/i386/i686/fpu/e_logl.S (__ieee754_logl): Likewise.
* sysdeps/x86_64/fpu/e_logl.S (__ieee754_logl): Likewise.
* math/libm-test.inc (log_test_data): Add sNaN tests.
| -rw-r--r-- | ChangeLog | 7 | ||||
| -rw-r--r-- | math/libm-test.inc | 2 | ||||
| -rw-r--r-- | sysdeps/i386/fpu/e_logl.S | 1 | ||||
| -rw-r--r-- | sysdeps/i386/i686/fpu/e_logl.S | 1 | ||||
| -rw-r--r-- | sysdeps/x86_64/fpu/e_logl.S | 1 |
5 files changed, 12 insertions, 0 deletions
@@ -1,5 +1,12 @@ 2016-06-08 Joseph Myers <joseph@codesourcery.com> + [BZ #20227] + * sysdeps/i386/fpu/e_logl.S (__ieee754_logl): Add NaN input to + itself. + * sysdeps/i386/i686/fpu/e_logl.S (__ieee754_logl): Likewise. + * sysdeps/x86_64/fpu/e_logl.S (__ieee754_logl): Likewise. + * math/libm-test.inc (log_test_data): Add sNaN tests. + [BZ #20226] * sysdeps/i386/fpu/e_expl.S (IEEE754_EXPL): Add NaN argument to itself. diff --git a/math/libm-test.inc b/math/libm-test.inc index 35a256c485..c16009b9ae 100644 --- a/math/libm-test.inc +++ b/math/libm-test.inc @@ -8998,6 +8998,8 @@ static const struct test_f_f_data log_test_data[] = TEST_f_f (log, plus_infty, plus_infty, ERRNO_UNCHANGED), TEST_f_f (log, qnan_value, qnan_value, NO_INEXACT_EXCEPTION|ERRNO_UNCHANGED), TEST_f_f (log, -qnan_value, qnan_value, NO_INEXACT_EXCEPTION|ERRNO_UNCHANGED), + TEST_f_f (log, snan_value, qnan_value, INVALID_EXCEPTION), + TEST_f_f (log, -snan_value, qnan_value, INVALID_EXCEPTION), AUTO_TESTS_f_f (log), }; diff --git a/sysdeps/i386/fpu/e_logl.S b/sysdeps/i386/fpu/e_logl.S index 7a6d370e8f..53127d704e 100644 --- a/sysdeps/i386/fpu/e_logl.S +++ b/sysdeps/i386/fpu/e_logl.S @@ -67,6 +67,7 @@ ENTRY(__ieee754_logl) 3: jp 4b // in case x is +-Inf fstp %st(1) fstp %st(1) + fadd %st(0) ret END (__ieee754_logl) diff --git a/sysdeps/i386/i686/fpu/e_logl.S b/sysdeps/i386/i686/fpu/e_logl.S index e9ecbcd472..7e3bc8d817 100644 --- a/sysdeps/i386/i686/fpu/e_logl.S +++ b/sysdeps/i386/i686/fpu/e_logl.S @@ -64,6 +64,7 @@ ENTRY(__ieee754_logl) ret 3: fstp %st(1) + fadd %st(0) ret END (__ieee754_logl) diff --git a/sysdeps/x86_64/fpu/e_logl.S b/sysdeps/x86_64/fpu/e_logl.S index dbe6fd59dc..0d3576f48b 100644 --- a/sysdeps/x86_64/fpu/e_logl.S +++ b/sysdeps/x86_64/fpu/e_logl.S @@ -66,6 +66,7 @@ ENTRY(__ieee754_logl) jnz 4b // in case x is +-Inf fstp %st(1) fstp %st(1) + fadd %st(0) ret END (__ieee754_logl) |