diff options
| author | Siddhesh Poyarekar <siddhesh@redhat.com> | 2014-08-13 14:05:15 +0530 |
|---|---|---|
| committer | Siddhesh Poyarekar <siddhesh@redhat.com> | 2014-08-13 14:05:16 +0530 |
| commit | 508ce3acd95e0782bc81e1f1eb84c43fa6978cfc (patch) | |
| tree | fe1327823360e8a139b867e1ad721cf41c5dc15e /ChangeLog | |
| parent | b4acef1ffe2e1ba6c608f31c1954a8100d3eabb0 (diff) | |
| download | glibc-508ce3acd95e0782bc81e1f1eb84c43fa6978cfc.tar.gz glibc-508ce3acd95e0782bc81e1f1eb84c43fa6978cfc.tar.xz glibc-508ce3acd95e0782bc81e1f1eb84c43fa6978cfc.zip | |
Disable x87 inline functions for x86_64 and SSE [BZ #17262]
Since:
commit 409e00bd69b8d8dd74d7327085351d26769ea6fc
Author: H.J. Lu <hjl.tools@gmail.com>
Date: Wed Jan 29 07:51:41 2014 -0800
Disable x87 inline functions for SSE2 math
When i386 and x86-64 mathinline.h was merged into a single mathinline.h,
"gcc -m32" enables x87 inline functions on x86-64 even when -mfpmath=sse
and SSE2 is enabled. It is a regression on x86-64. We should check
__SSE2_MATH__ instead of __x86_64__ when disabling x87 inline functions.
gcc-3.2 is unable to correctly compile x86_64 routines for llrint
since it gets redefined. This is because gcc 3.2 does not set
__SSE2_MATH__ for x86_64, thus exposing the duplicate definition.
The correct fix ought to be to check for both __SSE2_MATH__ and
__x86_64__ and enable those bits only when neither are defined.
Tested fix with the reproducer for
409e00bd69b8d8dd74d7327085351d26769ea6fc as well as with gcc-3.2.
Diffstat (limited to 'ChangeLog')
| -rw-r--r-- | ChangeLog | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog index e3a1f0a974..4508b54e82 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2014-08-13 Siddhesh Poyarekar <siddhesh@redhat.com> + + [BZ #17262] + * sysdeps/x86/fpu/bits/mathinline.h: Check both __SSE2_MATH__ + and __x86_64__ when disabling x87 inline functions. + 2014-08-12 H.J. Lu <hongjiu.lu@intel.com> [BZ #17259] |