Message ID | 20181222204809.1883-1-aurelien@aurel32.net |
---|---|
State | New |
Headers | show |
Series | [v2] Only build libm with -fno-math-errno | expand |
On Sat, 22 Dec 2018, Aurelien Jarno wrote: > v1 -> v2: improved the commit message with a better description of the > issue and the GCC bug number. No code change. I think the comment, not just the commit message should explain the issue (i.e., that only libnldbl nonlib testsuite should logically be using -fmath-errno, but everything apart from libm is currently using -fmath-errno because of that GCC bug). (I would say that the comment should give the GCC version in which the bug is fixed to make it obvious when we can change the code back, but that doesn't work when the GCC bug isn't yet fixed.) Could you add a testcase to the testsuite (that appropriately interposes malloc to demonstrate the bug as in the one attached to the bug, thus failing before the Makeconfig fix and passing after it)?
diff --git a/ChangeLog b/ChangeLog index 881808dead..a64ce96bea 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2018-12-22 Aurelien Jarno <aurelien@aurel32.net> + + [BZ #24024] + * Makeconfig: Build libm with -fno-math-errno but build the remaining + code with -fmath-errno. + 2018-12-21 Joseph Myers <joseph@codesourcery.com> [BZ #24023] diff --git a/Makeconfig b/Makeconfig index fbcf69e7c2..debdd3b357 100644 --- a/Makeconfig +++ b/Makeconfig @@ -831,8 +831,8 @@ endif # disable any optimization that assume default rounding mode. +math-flags = -frounding-math -# Build libc/libm using -fno-math-errno, but run testsuite with -fmath-errno. -+extra-math-flags = $(if $(filter libnldbl nonlib testsuite,$(in-module)),-fmath-errno,-fno-math-errno) +# Build libm using -fno-math-errno ++extra-math-flags = $(if $(filter libm,$(in-module)),-fno-math-errno,-fmath-errno) # We might want to compile with some stack-protection flag. ifneq ($(stack-protector),)