From patchwork Mon Dec 8 11:32:10 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andrew Senkevich X-Patchwork-Id: 4104 Received: (qmail 2979 invoked by alias); 8 Dec 2014 11:32:47 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 2890 invoked by uid 89); 8 Dec 2014 11:32:46 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.8 required=5.0 tests=AWL, BAYES_00, FREEMAIL_FROM, RCVD_IN_DNSWL_LOW, SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-lb0-f173.google.com X-Received: by 10.152.88.69 with SMTP id be5mr16066527lab.36.1418038361028; Mon, 08 Dec 2014 03:32:41 -0800 (PST) MIME-Version: 1.0 In-Reply-To: References: From: Andrew Senkevich Date: Mon, 8 Dec 2014 15:32:10 +0400 Message-ID: Subject: Re: [PATCH] Removal of duplication of -frounding-math for math tests To: Joseph Myers Cc: libc-alpha 2014-12-06 1:36 GMT+03:00 Joseph Myers : > 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-ldouble.c): Likewise. (CPPFLAGS-test-ifloat.c): Likewise. (CPPFLAGS-test-idouble.c): Likewise. (CPPFLAGS-test-ildoubl.c): Likewise. (CFLAGS-test-test-fenv.c): Remove variable. (CFLAGS-test-misc.c): Likewise. Ok for trunk? --- WBR, Andrew 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.