remove obsolete/non working ways to modify CFLAGS in memcheck/Makefile.am

memcheck/Makefile.am contains 2 CFLAGS modifications lines that
are not working.
Remove these confusing lines.

(I have checked that the proper flags are still used for the 2 involved files)

diff --git a/memcheck/Makefile.am b/memcheck/Makefile.am
index 476bc70dfa7954f6e561eeb26131c53a91a2979d..721cb21eb05d7871f4cfb95c0b5fe81f44d3a6f6 100644 (file)
--- a/memcheck/Makefile.am
+++ b/memcheck/Makefile.am
@@ -69,11 +69,6 @@ memcheck_@VGCONF_ARCH_SEC@_@VGCONF_OS@_LINK = \
        $(memcheck_@VGCONF_ARCH_SEC@_@VGCONF_OS@_LDFLAGS)
 endif
 
-# mc_main.c contains the helper function for memcheck that get called
-# all the time. To maximise performance compile with -fomit-frame-pointer
-# Primary beneficiary is x86.
-mc_main.o: CFLAGS += -fomit-frame-pointer
-
 #----------------------------------------------------------------------------
 # vgpreload_memcheck-<platform>.so
 #----------------------------------------------------------------------------
@@ -87,6 +82,8 @@ if VGCONF_OS_IS_DARWIN
 noinst_DSYMS = $(noinst_PROGRAMS)
 endif
 
+# mc_replace_strmem.c runs on the simulated CPU, and is built with
+# AM_CFLAGS_PSO_* (see $(top_srcdir)/Makefile.all.am).
 VGPRELOAD_MEMCHECK_SOURCES_COMMON = mc_replace_strmem.c
 
 vgpreload_memcheck_@VGCONF_ARCH_PRI@_@VGCONF_OS@_so_SOURCES      = \
@@ -115,10 +112,3 @@ vgpreload_memcheck_@VGCONF_ARCH_SEC@_@VGCONF_OS@_so_LDFLAGS      = \
        $(LIBREPLACEMALLOC_LDFLAGS_@VGCONF_PLATFORM_SEC_CAPS@)
 endif
 
-# mc_replace_strmem.c runs on the simulated CPU, and it often appears
-# in stack traces shown to the user.  It is built with
-# -fno-omit-frame-pointer so as to guarantee robust backtraces on x86,
-# on which CFI based unwinding is not the "normal" case and so is
-# sometimes fragile.
-mc_replace_strmem.o: CFLAGS += -fno-omit-frame-pointer
-