This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Include $(CFLAGS-$@) in ALL_CXXFLAGS


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  <joseph@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
joseph@codesourcery.com


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]