diff options
| author | Joseph Myers <joseph@codesourcery.com> | 2015-09-17 16:47:14 +0000 |
|---|---|---|
| committer | Joseph Myers <joseph@codesourcery.com> | 2015-09-17 16:47:14 +0000 |
| commit | b8682397ab2db1aed7f25d0a0c7c81134a97c8c7 (patch) | |
| tree | 76b6a9cbc7456f7ee53fe40cc86e0c7fe5e87f7a /sysdeps/ieee754/flt-32/s_finitef.c | |
| parent | 46f74e1deee549b41160d353ce0c8f7db555d36c (diff) | |
| download | glibc-b8682397ab2db1aed7f25d0a0c7c81134a97c8c7.tar.gz glibc-b8682397ab2db1aed7f25d0a0c7c81134a97c8c7.tar.xz glibc-b8682397ab2db1aed7f25d0a0c7c81134a97c8c7.zip | |
Reduce number of constants in __finite* (bug 15384).
Bug 15384 notes that in __finite, two different constants are used that could be the same constant (the result only depends on the exponent of the floating-point representation), and that using the same constant is better for architectures where constants need loading from a constant pool. This patch implements that change. Tested for x86_64, mips64 and powerpc. [BZ #15384] * sysdeps/ieee754/dbl-64/s_finite.c (FINITE): Use same constant as bit-mask as in subtraction. * sysdeps/ieee754/dbl-64/wordsize-64/s_finite.c (__finite): Likewise. * sysdeps/ieee754/flt-32/s_finitef.c (FINITEF): Likewise. * sysdeps/ieee754/ldbl-128/s_finitel.c (__finitel): Likewise. * sysdeps/ieee754/ldbl-128ibm/s_finitel.c (__finitel): Likewise.
Diffstat (limited to 'sysdeps/ieee754/flt-32/s_finitef.c')
| -rw-r--r-- | sysdeps/ieee754/flt-32/s_finitef.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sysdeps/ieee754/flt-32/s_finitef.c b/sysdeps/ieee754/flt-32/s_finitef.c index 4ea270ae07..4c5b339235 100644 --- a/sysdeps/ieee754/flt-32/s_finitef.c +++ b/sysdeps/ieee754/flt-32/s_finitef.c @@ -35,7 +35,7 @@ int FINITEF(float x) { int32_t ix; GET_FLOAT_WORD(ix,x); - return (int)((u_int32_t)((ix&0x7fffffff)-0x7f800000)>>31); + return (int)((u_int32_t)((ix&0x7f800000)-0x7f800000)>>31); } hidden_def (__finitef) weak_alias (__finitef, finitef) |