diff options
author | Joseph Myers <joseph@codesourcery.com> | 2015-10-07 00:40:12 +0000 |
---|---|---|
committer | Joseph Myers <joseph@codesourcery.com> | 2015-10-07 00:40:12 +0000 |
commit | b75bc69cdfe5247b2156ce249518f1c1df4d797d (patch) | |
tree | 7cbc51c7f8a7954e5394fcd873f42f7abeaa22df /sysdeps/x86_64 | |
parent | a68dec3b03b9c163ad79192cbe837208b2f90de1 (diff) | |
download | glibc-b75bc69cdfe5247b2156ce249518f1c1df4d797d.tar glibc-b75bc69cdfe5247b2156ce249518f1c1df4d797d.tar.gz glibc-b75bc69cdfe5247b2156ce249518f1c1df4d797d.tar.bz2 glibc-b75bc69cdfe5247b2156ce249518f1c1df4d797d.zip |
Don't use dbl-64/wordsize-64 lround based on llround for ILP32 (bug 19079).
The implementation of lround in dbl-64/wordsize-64 as an alias or
wrapper for llround is always incorrect when long is not 64-bit,
because it misses required exceptions in overflow cases, as shown by
my recently added tests. This patch removes that alias / wrapper in
the non-LP64 case, together with the REGISTER_CAST_INT32_TO_INT64
macro, restoring the previous version of lround for dbl-64/wordsize-64
(newly conditioned on !_LP64).
Tested for x86_64, and for mips64 with use of dbl-64/wordsize-64
enabled.
[BZ #19079]
* sysdeps/ieee754/dbl-64/wordsize-64/s_lround.c: Restore previous
file, conditioned on [!_LP64].
* sysdeps/ieee754/dbl-64/wordsize-64/s_llround.c
[!_LP64] (__lround): Do not define as function or alias.
[!_LP64] (lround): Likewise.
[!_LP64] (__lroundl): Likewise.
[!_LP64] (lroundl): Likewise.
* sysdeps/tile/sysdep.h (REGISTER_CAST_INT32_TO_INT64): Remove
macro.
* sysdeps/x86_64/x32/sysdep.h (REGISTER_CAST_INT32_TO_INT64):
Likewise.
Diffstat (limited to 'sysdeps/x86_64')
-rw-r--r-- | sysdeps/x86_64/x32/sysdep.h | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/sysdeps/x86_64/x32/sysdep.h b/sysdeps/x86_64/x32/sysdep.h index 0cbc1a083f..9b8700522f 100644 --- a/sysdeps/x86_64/x32/sysdep.h +++ b/sysdeps/x86_64/x32/sysdep.h @@ -90,7 +90,3 @@ # define R15_LP "r15d" #endif /* __ASSEMBLER__ */ - -/* On x32, it is not required to normalize a 64-bit value before using - it as a 32-bit value. */ -#define REGISTER_CAST_INT32_TO_INT64 0 |