diff options
| author | Joseph Myers <joseph@codesourcery.com> | 2015-09-23 22:52:13 +0000 |
|---|---|---|
| committer | Joseph Myers <joseph@codesourcery.com> | 2015-09-23 22:52:13 +0000 |
| commit | cfcbebf7ac5de5bf1928201b6bd9cd5e6764b4a7 (patch) | |
| tree | 865093eaae5893910d88d19d38a4e36206a988c1 | |
| parent | d96164c33012fccc7ba3ebb4d324c7fd0c6b5836 (diff) | |
| download | glibc-cfcbebf7ac5de5bf1928201b6bd9cd5e6764b4a7.tar.gz | |
Don't use volatile in exp2f.
sysdeps/ieee754/flt-32/e_exp2f.c declares two variable as "static
const volatile float". Maybe this use of "volatile" was originally
intended to inhibit optimization of underflowing / overflowing
operations such as TWOM100 * TWOM100; in any case, it's not currently
needed, as given -frounding-math constant folding of such expressions
is properly disabled when it would be unsafe. This patch removes the
unnecessary use of "volatile".
Tested for x86_64.
* sysdeps/ieee754/flt-32/e_exp2f.c (TWOM100): Remove volatile.
(TWO127): Likewise.
| -rw-r--r-- | ChangeLog | 3 | ||||
| -rw-r--r-- | sysdeps/ieee754/flt-32/e_exp2f.c | 4 |
2 files changed, 5 insertions, 2 deletions
@@ -1,5 +1,8 @@ 2015-09-23 Joseph Myers <joseph@codesourcery.com> + * sysdeps/ieee754/flt-32/e_exp2f.c (TWOM100): Remove volatile. + (TWO127): Likewise. + * sysdeps/generic/math_private.h (fabs_tg): New macro. (min_of_type): Likewise. (math_check_force_underflow): Likewise. diff --git a/sysdeps/ieee754/flt-32/e_exp2f.c b/sysdeps/ieee754/flt-32/e_exp2f.c index d8e7a9c8bc..c053340310 100644 --- a/sysdeps/ieee754/flt-32/e_exp2f.c +++ b/sysdeps/ieee754/flt-32/e_exp2f.c @@ -37,8 +37,8 @@ #include "t_exp2f.h" -static const volatile float TWOM100 = 7.88860905e-31; -static const volatile float TWO127 = 1.7014118346e+38; +static const float TWOM100 = 7.88860905e-31; +static const float TWO127 = 1.7014118346e+38; float __ieee754_exp2f (float x) |