diff options
| author | Joseph Myers <joseph@codesourcery.com> | 2015-09-18 20:00:48 +0000 |
|---|---|---|
| committer | Joseph Myers <joseph@codesourcery.com> | 2015-09-18 20:00:48 +0000 |
| commit | c8235dda728c4452f57c4c0d1bccc4e9a67da80c (patch) | |
| tree | ae521c94b4648e089ee442cf8991963e7ffd97fe /ChangeLog | |
| parent | fe8c2b33aed067282ecac0cc3fdff1feef88548b (diff) | |
| download | glibc-c8235dda728c4452f57c4c0d1bccc4e9a67da80c.tar.gz glibc-c8235dda728c4452f57c4c0d1bccc4e9a67da80c.tar.xz glibc-c8235dda728c4452f57c4c0d1bccc4e9a67da80c.zip | |
Avoid excess range overflowing results from cosh, sinh, lgamma (bug 18980).
Various i386 libm functions return values with excess range and precision; Wilco Dijkstra's patches to make isfinite etc. expand inline cause this pre-existing issue to result in test failures (when e.g. a result that overflows float but not long double gets counted as overflowing for some purposes but not others). This patch addresses those cases arising from functions defined in C, adding a math_narrow_eval macro that forces values to memory to eliminate excess precision if FLT_EVAL_METHOD indicates this is needed, and is a no-op otherwise. I'll convert existing uses of volatile and asm for this purpose to use the new macro later, once i386 has clean test results again (which requires fixes for .S files as well). Tested for x86_64 and x86. Committed. [BZ #18980] * sysdeps/generic/math_private.h: Include <float.h>. (math_narrow_eval): New macro. [FLT_EVAL_METHOD != 0] (excess_precision): Likewise. * sysdeps/ieee754/dbl-64/e_cosh.c (__ieee754_cosh): Use math_narrow_eval on overflowing return value. * sysdeps/ieee754/dbl-64/e_lgamma_r.c (__ieee754_lgamma_r): Likewise. * sysdeps/ieee754/dbl-64/e_sinh.c (__ieee754_sinh): Likewise. * sysdeps/ieee754/flt-32/e_coshf.c (__ieee754_coshf): Likewise. * sysdeps/ieee754/flt-32/e_lgammaf_r.c (__ieee754_lgammaf_r): Likewise. * sysdeps/ieee754/flt-32/e_sinhf.c (__ieee754_sinhf): Likewise.
Diffstat (limited to 'ChangeLog')
| -rw-r--r-- | ChangeLog | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog index 9a89e9d9e5..57b07b36ee 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,19 @@ +2015-09-18 Joseph Myers <joseph@codesourcery.com> + + [BZ #18980] + * sysdeps/generic/math_private.h: Include <float.h>. + (math_narrow_eval): New macro. + [FLT_EVAL_METHOD != 0] (excess_precision): Likewise. + * sysdeps/ieee754/dbl-64/e_cosh.c (__ieee754_cosh): Use + math_narrow_eval on overflowing return value. + * sysdeps/ieee754/dbl-64/e_lgamma_r.c (__ieee754_lgamma_r): + Likewise. + * sysdeps/ieee754/dbl-64/e_sinh.c (__ieee754_sinh): Likewise. + * sysdeps/ieee754/flt-32/e_coshf.c (__ieee754_coshf): Likewise. + * sysdeps/ieee754/flt-32/e_lgammaf_r.c (__ieee754_lgammaf_r): + Likewise. + * sysdeps/ieee754/flt-32/e_sinhf.c (__ieee754_sinhf): Likewise. + 2015-09-18 Wilco Dijkstra <wdijkstr@arm.com> * include/math.h: Remove __isinf_ns, __isinf_nsf, __isinf_nsl. |