Add -Wreorder to the list of C++ warnings.
authorEric Christopher <echristo@gmail.com>
Tue, 14 May 2013 21:49:38 +0000 (21:49 +0000)
committerEric Christopher <echristo@gmail.com>
Tue, 14 May 2013 21:49:38 +0000 (21:49 +0000)
This built clean with clang, but if we see false positives on the bots
then we'll revert and turn it into a compiler specific check.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@181836 91177308-0d34-0410-b5e6-96231b3b80d8

Makefile.rules

index f0c542b7f82cd28432101bd1caa72db22cfc5b54..396262aad2ec8e67a0e0099e89dfa3b12b28afbd 100644 (file)
@@ -674,7 +674,8 @@ CompileCommonOpts += -Wall -W -Wno-unused-parameter -Wwrite-strings \
                      $(NO_UNINITIALIZED) $(NO_MAYBE_UNINITIALIZED) \
                      $(NO_MISSING_FIELD_INITIALIZERS)
 # Enable cast-qual for C++; the workaround is to use const_cast.
-CXX.Flags += -Wcast-qual
+# Also enable -Wreorder.
+CXX.Flags += -Wcast-qual -Wreorder
 
 ifeq ($(HOST_OS),HP-UX)
   CompileCommonOpts := -D_REENTRANT -D_HPUX_SOURCE