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 30c25352757c0d1908b497482b49e0c56034b7c1..0acb2d7ecbad8dc4cfb1bdd7f58019c4a5d7000a 100644 (file)
--- 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 4a9d93d7e4ad1d27647ef23f49a70fba6cedf50c..0da581f5e6a95d8a7e0847cad53418b4bbb35114 100644 (file)
--- 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      = \