Compile with -O2 even if debug=yes. Suggestion from Yavor.

git-svn-id: svn+ssh://svn.gna.org/svn/gnustep/tools/make/trunk@28461 72102866-910b-0410-8b05-ffd578937521

ChangeLog

@@ -1,3 +1,11 @@
+2009-08-14  Nicola Pero  <nicola.pero@meta-innovation.com>
+	
+	* common.make: Do not filter out the -Ox flags when debug=yes.
+	This means debug=yes now uses "-g -O2" by default - instead of
+	just "-g" (Suggestion by Yavor Doganov <yavor@gnu.org>).
+	* Documentation/releasenotes.texi: Updated.
+	* RELEASENOTES: Regenerated.
+
 2009-05-09  Adam Fedor  <fedor@gnu.org>
 
 	* Version 2.2.0

Documentation/releasenotes.texi

@@ -4,6 +4,19 @@ The release notes include descriptions of API changes, behavior
 changes and other information that might help developers and users
 migrate to using a newer version of the make system.
 
+@section Version 2.2.1
+@table @samp
+
+@item debug=yes no longer strips -O2
+gnustep-make no longer strips the -Ox optimization flag when compiling
+C/Objective-C/C++/Objective-C++ code with debug=yes; as a consequence,
+on most platforms debug=yes (which is the default) now builds using -g
+-O2 instead of just -g.  If you do not want the -O2 flag, you can
+override the OPTFLAG variable on the make command line (or in your
+GNUmakefile) as in 'make OPTFLAG='.
+
+@end table
+
 @section Version 2.2.0
 @table @samp
 

RELEASENOTES

@@ -5,7 +5,19 @@ The release notes include descriptions of API changes, behavior changes
 and other information that might help developers and users migrate to
 using a newer version of the make system.
 
-1.1 Version 2.2.0
+1.1 Version 2.2.1
+=================
+
+`debug=yes no longer strips -O2'
+     gnustep-make no longer strips the -Ox optimization flag when
+     compiling C/Objective-C/C++/Objective-C++ code with debug=yes; as
+     a consequence, on most platforms debug=yes (which is the default)
+     now builds using -g -O2 instead of just -g.  If you do not want
+     the -O2 flag, you can override the OPTFLAG variable on the make
+     command line (or in your GNUmakefile) as in 'make OPTFLAG='.
+
+
+1.2 Version 2.2.0
 =================
 
 `libobjc library'
@@ -31,7 +43,7 @@ using a newer version of the make system.
      at removing directories that were created during by 'make install'.
 
 
-1.2 Version 2.0.7
+1.3 Version 2.0.7
 =================
 
 `default installation'
@@ -73,7 +85,7 @@ using a newer version of the make system.
      default.
 
 
-1.3 Version 2.0.6
+1.4 Version 2.0.6
 =================
 
 `GNUSTEP_ABSOLUTE_INSTALL_PATHS'
@@ -101,7 +113,7 @@ using a newer version of the make system.
      go in that subdirectory and install it.
 
 
-1.4 Version 2.0.5
+1.5 Version 2.0.5
 =================
 
 `default filesystem layout on apple-apple-apple'
@@ -146,7 +158,7 @@ using a newer version of the make system.
      variable (see above).
 
 
-1.5 Version 2.0.0
+1.6 Version 2.0.0
 =================
 
 Version 2.0.0 is a new major release of gnustep-make which includes a

common.make

@@ -652,8 +652,6 @@ ifneq ($(debug), no)
 endif
 
 ifeq ($(debug), yes)
-  # This is filtered out as it compromised debugging
-  OPTFLAG := $(filter-out -O%, $(OPTFLAG))
   ADDITIONAL_FLAGS += -g -Wall -DDEBUG -fno-omit-frame-pointer
   INTERNAL_JAVACFLAGS += -g -deprecation
 else

config.make.in

@@ -31,6 +31,11 @@ MAKE_WITH_INFO_FUNCTION = @MAKE_WITH_INFO_FUNCTION@
 # Binary and compile tools
 #
 CC       = @CC@
+# TODO: Because of the following, OPTFLAG usually ends up being '-g
+# -O2'.  The '-g' is fairly harmless as you can always use strip=yes
+# which will strip the object files upon installation; still, it's not
+# very elegant since -g is already added elsewhere for debug=yes, and
+# it ends up appearing twice in the gcc command-line.
 OPTFLAG  = @CFLAGS@
 OBJCFLAGS= @OBJCFLAGS@
 OBJC_LIB_FLAG = @OBJC_LIB_FLAG@