I noticed that in gcc/Makefile.in, ALL_CFLAGS included $(CFLAGS-$@)
but ALL_CXXFLAGS was missing it.  I don't see any reason for this
difference; this patch makes the two agree more closely.

Bootstrapped with no regressions on x86_64-unknown-linux-gnu.  OK to
commit?

2011-08-23  Joseph Myers  <jos...@codesourcery.com>

        * Makefile.in (ALL_CXXFLAGS): Include $(CFLAGS-$@).

Index: Makefile.in
===================================================================
--- Makefile.in (revision 177999)
+++ Makefile.in (working copy)
@@ -1029,7 +1029,7 @@
   $(CFLAGS) $(INTERNAL_CFLAGS) $(COVERAGE_FLAGS) $(WARN_CFLAGS) @DEFS@
 
 # The C++ version.
-ALL_CXXFLAGS = $(T_CFLAGS) $(CXXFLAGS) $(INTERNAL_CFLAGS) \
+ALL_CXXFLAGS = $(T_CFLAGS) $(CFLAGS-$@) $(CXXFLAGS) $(INTERNAL_CFLAGS) \
   $(COVERAGE_FLAGS) $(WARN_CXXFLAGS) @DEFS@
 
 # Likewise.  Put INCLUDES at the beginning: this way, if some autoconf macro

-- 
Joseph S. Myers
jos...@codesourcery.com

Reply via email to