From: Joseph Myers <joseph@codesourcery.com>
Date: Thu, 20 Aug 2015 18:00:09 +0000 (+0000)
Subject: Don't use -Wno-uninitialized in math/.
X-Git-Tag: glibc-2.23~617
X-Git-Url: https://sourceware.org/git/?a=commitdiff_plain;h=772e741ba5afede4892078102a620e30aeac0c87;p=glibc.git

Don't use -Wno-uninitialized in math/.

The uninitialized variable warnings in math/ having been fixed for all
the supported floating-point formats, this patch removes the use of
-Wno-uninitialized there, continuing with the goal of avoiding -Wno-
options in makefiles as far as possible..

Tested for x86_64 and x86 (full build and testsuite runs), and for
powerpc and mips64 (verified that glibc builds without errors).

	* math/Makefile (CFLAGS): Don't add -Wno-uninitialized.
---

diff --git a/ChangeLog b/ChangeLog
index 93d556311e..73f16c4528 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,7 @@
 2015-08-20  Joseph Myers  <joseph@codesourcery.com>
 
+	* math/Makefile (CFLAGS): Don't add -Wno-uninitialized.
+
 	* sysdeps/ieee754/ldbl-128ibm/s_nearbyintl.c: Always initialize
 	variables for high and low parts before possibly modifying them.
 
diff --git a/math/Makefile b/math/Makefile
index d3b483d8a9..c98c3c4d94 100644
--- a/math/Makefile
+++ b/math/Makefile
@@ -249,9 +249,6 @@ ifneq ($(long-double-fcts),yes)
 math-CPPFLAGS += -DNO_LONG_DOUBLE -D_Mlong_double_=double
 endif
 
-# The fdlibm code generates a lot of these warnings but is otherwise clean.
-override CFLAGS += -Wno-uninitialized
-
 # The -lieee library is actually an object file.
 # The module just defines the _LIB_VERSION_ variable.
 # It's not a library to make sure it is linked in instead of s_lib_version.o.