Skip to content

Commit

Permalink
Add -Wreorder to the list of C++ warnings.
Browse files Browse the repository at this point in the history
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
  • Loading branch information
echristo committed May 14, 2013
1 parent 4d7f2ce commit f8f7086
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion Makefile.rules
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit f8f7086

Please sign in to comment.