aboutsummaryrefslogtreecommitdiffstats
path: root/meta/recipes-devtools/valgrind
diff options
context:
space:
mode:
authorAndre McCurdy <armccurdy@gmail.com>2016-01-19 18:47:49 -0800
committerRichard Purdie <richard.purdie@linuxfoundation.org>2016-01-22 23:42:37 +0000
commit1c8ed8dada8d3399bd825d6f34fc440ce793ec4e (patch)
treed095f0160feb9e64cca9534c8cfccb737a08124a /meta/recipes-devtools/valgrind
parent99d4d97f51b658d58a50789056b422e48df89ad9 (diff)
downloadopenembedded-core-1c8ed8dada8d3399bd825d6f34fc440ce793ec4e.tar.gz
valgrind: let valgrind determine its own optimisation flags
Valgrind likes to control its own optimisation flags. It generally defaults to -O2 but uses -O0 for some specific test apps etc. Passing our own flags (via CFLAGS) means we interfere with that. Giving valgrind control of optimisation is hopefully an even better solution than the previous one of forcing -O0 for all tests. http://git.openembedded.org/openembedded-core/commit/?h=master-next&id=98c4a3ffb8dca10739be600e8d6df7fb6aa4958f Signed-off-by: Andre McCurdy <armccurdy@gmail.com> Signed-off-by: Ross Burton <ross.burton@intel.com>
Diffstat (limited to 'meta/recipes-devtools/valgrind')
-rw-r--r--meta/recipes-devtools/valgrind/valgrind_3.11.0.bb9
1 files changed, 6 insertions, 3 deletions
diff --git a/meta/recipes-devtools/valgrind/valgrind_3.11.0.bb b/meta/recipes-devtools/valgrind/valgrind_3.11.0.bb
index 70d84fe9a2..8e9b72cd61 100644
--- a/meta/recipes-devtools/valgrind/valgrind_3.11.0.bb
+++ b/meta/recipes-devtools/valgrind/valgrind_3.11.0.bb
@@ -44,6 +44,11 @@ EXTRA_OECONF_append_arm = " --host=armv7${HOST_VENDOR}-${HOST_OS}"
EXTRA_OEMAKE = "-w"
+# valgrind likes to control its own optimisation flags. It generally defaults
+# to -O2 but uses -O0 for some specific test apps etc. Passing our own flags
+# (via CFLAGS) means we interfere with that.
+SELECTED_OPTIMIZATION = ""
+
CFLAGS_append_libc-uclibc = " -D__UCLIBC__ "
do_install_append () {
@@ -63,10 +68,9 @@ RDEPENDS_${PN}-ptest += " sed perl glibc-utils perl-module-file-glob"
INSANE_SKIP_${PN}-ptest += "file-rdeps"
do_compile_ptest() {
- oe_runmake check CFLAGS="${CFLAGS} -O0" CXXFLAGS="${CXXFLAGS} -O0"
+ oe_runmake check
}
-
do_install_ptest() {
chmod +x ${B}/tests/vg_regtest
@@ -107,4 +111,3 @@ do_install_ptest() {
# handle multilib
sed -i s:@libdir@:${libdir}:g ${D}${PTEST_PATH}/run-ptest
}
-