Removal of duplication of -frounding-math for math tests
diff mbox

Message ID CAMXFM3v7Y9iHcVsXa9Fj3pSSt+BpMFw2DAJU3mydtAAWbYGUeg@mail.gmail.com
State New
Headers show

Commit Message

Andrew Senkevich Dec. 5, 2014, 7:47 p.m. UTC
This patch is removal of duplication of -frounding-math for math tests
because it comes from math-flags in Makeconfig.

ChangeLog

        * math/Makefile: duplicated -frounding-math removed.


Is it ok for trunk?


--
WBR,
Andrew

Comments

Joseph Myers Dec. 5, 2014, 10:36 p.m. UTC | #1
On Fri, 5 Dec 2014, Andrew Senkevich wrote:

> This patch is removal of duplication of -frounding-math for math tests
> because it comes from math-flags in Makeconfig.
> 
> ChangeLog
> 
>         * math/Makefile: duplicated -frounding-math removed.

OK with a proper ChangeLog entry (which means naming each variable 
changed, along the lines of

	* math/Makefile (CFLAGS-test-float.c): Remove -frounding-math.
	(CFLAGS-test-double.c): Likewise.
	...
	(CFLAGS-test-test-fenv.c): Remove variable.
	...

).

Patch
diff mbox

diff --git a/math/Makefile b/math/Makefile
index 866bc0f..9ec450b 100644
--- a/math/Makefile
+++ b/math/Makefile
@@ -126,9 +126,9 @@  $(objpfx)test-ldouble.o: $(objpfx)libm-test.stmp
 $(objpfx)test-ildoubl.o: $(objpfx)libm-test.stmp
 endif

-CFLAGS-test-float.c = -fno-inline -ffloat-store -fno-builtin -frounding-math
-CFLAGS-test-double.c = -fno-inline -ffloat-store -fno-builtin -frounding-math
-CFLAGS-test-ldouble.c = -fno-inline -ffloat-store -fno-builtin -frounding-math
+CFLAGS-test-float.c = -fno-inline -ffloat-store -fno-builtin
+CFLAGS-test-double.c = -fno-inline -ffloat-store -fno-builtin
+CFLAGS-test-ldouble.c = -fno-inline -ffloat-store -fno-builtin
 CFLAGS-test-tgmath.c = -fno-builtin
 # The following testcase uses very long lines (>3 million), so it may take a
 # while to compile it. See: http://llvm.org/bugs/show_bug.cgi?id=14106 and
@@ -136,15 +136,12 @@  CFLAGS-test-tgmath.c = -fno-builtin
 CFLAGS-test-tgmath2.c = -fno-builtin
 CFLAGS-test-tgmath-ret.c = -fno-builtin
 CFLAGS-test-powl.c = -fno-builtin
-CFLAGS-test-test-fenv.c = -frounding-math
-CFLAGS-test-misc.c = -frounding-math
 CPPFLAGS-test-ifloat.c = -U__LIBC_INTERNAL_MATH_INLINES -D__FAST_MATH__ \
-                        -DTEST_FAST_MATH -fno-builtin -frounding-math
+                        -DTEST_FAST_MATH -fno-builtin
 CPPFLAGS-test-idouble.c = -U__LIBC_INTERNAL_MATH_INLINES -D__FAST_MATH__ \
-                        -DTEST_FAST_MATH -fno-builtin -frounding-math
+                        -DTEST_FAST_MATH -fno-builtin
 CPPFLAGS-test-ildoubl.c = -U__LIBC_INTERNAL_MATH_INLINES -D__FAST_MATH__ \
-                         -DTEST_FAST_MATH -fno-builtin -frounding-math
-
+                         -DTEST_FAST_MATH -fno-builtin

 # The -lieee module sets the _LIB_VERSION_ switch to IEEE mode
 # for error handling in the -lm functions.