aboutsummaryrefslogtreecommitdiff
path: root/sysdeps/i386
diff options
context:
space:
mode:
authorJoseph Myers <joseph@codesourcery.com>2021-09-21 21:54:37 +0000
committerJoseph Myers <joseph@codesourcery.com>2021-09-21 21:54:37 +0000
commit1356f38df5be0776823eb2c40cc4e607b86b9680 (patch)
tree52590f89365894844fac4bb98898e4725445c844 /sysdeps/i386
parent0b5ca7c3e551e5502f3be3b06453324fe8604e82 (diff)
downloadglibc-1356f38df5be0776823eb2c40cc4e607b86b9680.zip
glibc-1356f38df5be0776823eb2c40cc4e607b86b9680.tar.gz
glibc-1356f38df5be0776823eb2c40cc4e607b86b9680.tar.bz2
Fix f64xdivf128, f64xmulf128 spurious underflows (bug 28358)
As described in bug 28358, the round-to-odd computations used in the libm functions that round their results to a narrower format can yield spurious underflow exceptions in the following circumstances: the narrowing only narrows the precision of the type and not the exponent range (i.e., it's narrowing _Float128 to _Float64x on x86_64, x86 or ia64), the architecture does after-rounding tininess detection (which applies to all those architectures), the result is inexact, tiny before rounding but not tiny after rounding (with the chosen rounding mode) for _Float64x (which is possible for narrowing mul, div and fma, not for narrowing add, sub or sqrt), so the underflow exception resulting from the toward-zero computation in _Float128 is spurious for _Float64x. Fixed by making ROUND_TO_ODD call feclearexcept (FE_UNDERFLOW) in the problem cases (as indicated by an extra argument to the macro); there is never any need to preserve underflow exceptions from this part of the computation, because the conversion of the round-to-odd value to the narrower type will underflow in exactly the cases in which the function should raise that exception, but it may be more efficient to avoid the extra manipulation of the floating-point environment when not needed. Tested for x86_64 and x86, and with build-many-glibcs.py.
Diffstat (limited to 'sysdeps/i386')
-rw-r--r--sysdeps/i386/fpu/s_f32xdivf64.c2
-rw-r--r--sysdeps/i386/fpu/s_f32xmulf64.c2
2 files changed, 2 insertions, 2 deletions
diff --git a/sysdeps/i386/fpu/s_f32xdivf64.c b/sysdeps/i386/fpu/s_f32xdivf64.c
index 114a49b..be0adec 100644
--- a/sysdeps/i386/fpu/s_f32xdivf64.c
+++ b/sysdeps/i386/fpu/s_f32xdivf64.c
@@ -24,6 +24,6 @@ __f32xdivf64 (_Float64 x, _Float64 y)
{
/* To avoid double rounding, use round-to-odd on long double. */
NARROW_DIV_ROUND_TO_ODD ((long double) x, (long double) y, double,
- union ieee854_long_double, l, mantissa1);
+ union ieee854_long_double, l, mantissa1, false);
}
libm_alias_float32x_float64 (div)
diff --git a/sysdeps/i386/fpu/s_f32xmulf64.c b/sysdeps/i386/fpu/s_f32xmulf64.c
index 9636bf0..82ffff7 100644
--- a/sysdeps/i386/fpu/s_f32xmulf64.c
+++ b/sysdeps/i386/fpu/s_f32xmulf64.c
@@ -24,6 +24,6 @@ __f32xmulf64 (_Float64 x, _Float64 y)
{
/* To avoid double rounding, use round-to-odd on long double. */
NARROW_MUL_ROUND_TO_ODD ((long double) x, (long double) y, double,
- union ieee854_long_double, l, mantissa1);
+ union ieee854_long_double, l, mantissa1, false);
}
libm_alias_float32x_float64 (mul)