From: Julian Seward <jseward@acm.org>
Date: Fri, 16 Nov 2007 10:05:47 +0000 (+0000)
Subject: Change the baseline optimisation level from -O to -O2.
X-Git-Tag: svn/VALGRIND_3_3_0~129
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=40eb3fba0f5c405d6efb8f819753b2f78cc1bdd1;p=thirdparty%2Fvalgrind.git

Change the baseline optimisation level from -O to -O2.


git-svn-id: svn://svn.valgrind.org/valgrind/trunk@7162
---

diff --git a/Makefile.flags.am b/Makefile.flags.am
index 30c2535275..0acb2d7ecb 100644
--- a/Makefile.flags.am
+++ b/Makefile.flags.am
@@ -1,9 +1,14 @@
 
-# Baseline flags for all compilations
-AM_CFLAGS_BASE = -O -g -Wmissing-prototypes -Winline -Wall -Wshadow \
+# Baseline flags for all compilations.  Aim here is to maximise
+# performance and get whatever useful warnings we can out of gcc.
+AM_CFLAGS_BASE = -O2 -g -Wmissing-prototypes -Winline -Wall -Wshadow \
                  -Wpointer-arith -Wstrict-prototypes -Wmissing-declarations
 
-AM_CFLAGS_PIC = -fpic -fno-omit-frame-pointer
+# These flags are used for building the preload shared objects.
+# The aim is to give reasonable performance but also to have good
+# stack traces, since users often see stack traces extending 
+# into (and through) the preloads.
+AM_CFLAGS_PIC = -O -g -fpic -fno-omit-frame-pointer
 
 # Flags for specific targets
 AM_FLAG_M3264_X86_LINUX   = @FLAG_M32@
diff --git a/helgrind/Makefile.am b/helgrind/Makefile.am
index 4a9d93d7e4..0da581f5e6 100644
--- a/helgrind/Makefile.am
+++ b/helgrind/Makefile.am
@@ -23,7 +23,7 @@ endif
 VGPRELOAD_HELGRIND_SOURCES_COMMON = hg_intercepts.c
 
 vgpreload_helgrind_x86_linux_so_SOURCES      = $(VGPRELOAD_HELGRIND_SOURCES_COMMON)
-vgpreload_helgrind_x86_linux_so_CPPFLAGS     = $(AM_CPPFLAGS_X86_LINUX) -O -g -fno-omit-frame-pointer
+vgpreload_helgrind_x86_linux_so_CPPFLAGS     = $(AM_CPPFLAGS_X86_LINUX)
 vgpreload_helgrind_x86_linux_so_CFLAGS       = $(AM_CFLAGS_X86_LINUX) $(AM_CFLAGS_PIC)
 vgpreload_helgrind_x86_linux_so_DEPENDENCIES = $(LIBREPLACEMALLOC_X86_LINUX)
 vgpreload_helgrind_x86_linux_so_LDFLAGS      = \
@@ -31,7 +31,7 @@ vgpreload_helgrind_x86_linux_so_LDFLAGS      = \
 	$(LIBREPLACEMALLOC_LDFLAGS_X86_LINUX)
 
 vgpreload_helgrind_amd64_linux_so_SOURCES      = $(VGPRELOAD_HELGRIND_SOURCES_COMMON)
-vgpreload_helgrind_amd64_linux_so_CPPFLAGS     = $(AM_CPPFLAGS_AMD64_LINUX) -O -g
+vgpreload_helgrind_amd64_linux_so_CPPFLAGS     = $(AM_CPPFLAGS_AMD64_LINUX)
 vgpreload_helgrind_amd64_linux_so_CFLAGS       = $(AM_CFLAGS_AMD64_LINUX) $(AM_CFLAGS_PIC)
 vgpreload_helgrind_amd64_linux_so_DEPENDENCIES = $(LIBREPLACEMALLOC_AMD64_LINUX)
 vgpreload_helgrind_amd64_linux_so_LDFLAGS      = \
@@ -39,7 +39,7 @@ vgpreload_helgrind_amd64_linux_so_LDFLAGS      = \
 	$(LIBREPLACEMALLOC_LDFLAGS_AMD64_LINUX)
 
 vgpreload_helgrind_ppc32_linux_so_SOURCES      = $(VGPRELOAD_HELGRIND_SOURCES_COMMON)
-vgpreload_helgrind_ppc32_linux_so_CPPFLAGS     = $(AM_CPPFLAGS_PPC32_LINUX) -O -g
+vgpreload_helgrind_ppc32_linux_so_CPPFLAGS     = $(AM_CPPFLAGS_PPC32_LINUX)
 vgpreload_helgrind_ppc32_linux_so_CFLAGS       = $(AM_CFLAGS_PPC32_LINUX) $(AM_CFLAGS_PIC)
 vgpreload_helgrind_ppc32_linux_so_DEPENDENCIES = $(LIBREPLACEMALLOC_PPC32_LINUX)
 vgpreload_helgrind_ppc32_linux_so_LDFLAGS      = \
@@ -47,7 +47,7 @@ vgpreload_helgrind_ppc32_linux_so_LDFLAGS      = \
 	$(LIBREPLACEMALLOC_LDFLAGS_PPC32_LINUX)
 
 vgpreload_helgrind_ppc64_linux_so_SOURCES      = $(VGPRELOAD_HELGRIND_SOURCES_COMMON)
-vgpreload_helgrind_ppc64_linux_so_CPPFLAGS     = $(AM_CPPFLAGS_PPC64_LINUX) -O -g
+vgpreload_helgrind_ppc64_linux_so_CPPFLAGS     = $(AM_CPPFLAGS_PPC64_LINUX)
 vgpreload_helgrind_ppc64_linux_so_CFLAGS       = $(AM_CFLAGS_PPC64_LINUX) $(AM_CFLAGS_PIC)
 vgpreload_helgrind_ppc64_linux_so_DEPENDENCIES = $(LIBREPLACEMALLOC_PPC64_LINUX)
 vgpreload_helgrind_ppc64_linux_so_LDFLAGS      = \
@@ -55,7 +55,7 @@ vgpreload_helgrind_ppc64_linux_so_LDFLAGS      = \
 	$(LIBREPLACEMALLOC_LDFLAGS_PPC64_LINUX)
 
 vgpreload_helgrind_ppc32_aix5_so_SOURCES      = $(VGPRELOAD_HELGRIND_SOURCES_COMMON)
-vgpreload_helgrind_ppc32_aix5_so_CPPFLAGS     = $(AM_CPPFLAGS_PPC32_AIX5) -O -g
+vgpreload_helgrind_ppc32_aix5_so_CPPFLAGS     = $(AM_CPPFLAGS_PPC32_AIX5)
 vgpreload_helgrind_ppc32_aix5_so_CFLAGS       = $(AM_CFLAGS_PPC32_AIX5) $(AM_CFLAGS_PIC)
 vgpreload_helgrind_ppc32_aix5_so_DEPENDENCIES = $(LIBREPLACEMALLOC_PPC32_AIX5)
 vgpreload_helgrind_ppc32_aix5_so_LDFLAGS      = \
@@ -63,7 +63,7 @@ vgpreload_helgrind_ppc32_aix5_so_LDFLAGS      = \
 	$(LIBREPLACEMALLOC_LDFLAGS_PPC32_AIX5)
 
 vgpreload_helgrind_ppc64_aix5_so_SOURCES      = $(VGPRELOAD_HELGRIND_SOURCES_COMMON)
-vgpreload_helgrind_ppc64_aix5_so_CPPFLAGS     = $(AM_CPPFLAGS_PPC64_AIX5) -O -g
+vgpreload_helgrind_ppc64_aix5_so_CPPFLAGS     = $(AM_CPPFLAGS_PPC64_AIX5)
 vgpreload_helgrind_ppc64_aix5_so_CFLAGS       = $(AM_CFLAGS_PPC64_AIX5) $(AM_CFLAGS_PIC)
 vgpreload_helgrind_ppc64_aix5_so_DEPENDENCIES = $(LIBREPLACEMALLOC_PPC64_AIX5)
 vgpreload_helgrind_ppc64_aix5_so_LDFLAGS      = \