summaryrefslogtreecommitdiffstats
path: root/meta/recipes-support/boost/files/0001-build.sh-use-DNDEBUG-also-in-debug-builds.patch
diff options
context:
space:
mode:
authorRoss Burton <ross@burtonini.com>2020-09-29 23:16:37 +0100
committerRichard Purdie <richard.purdie@linuxfoundation.org>2020-10-06 23:14:24 +0100
commite6436043ca1fe160c09143d901e43b4d83d43d3f (patch)
tree0532532deeac87d9c029dcd48635841f19a32cb6 /meta/recipes-support/boost/files/0001-build.sh-use-DNDEBUG-also-in-debug-builds.patch
parente65267d3768429a39967de3c283c981a5848913e (diff)
downloadpoky-e6436043ca1fe160c09143d901e43b4d83d43d3f.tar.gz
bjam-native: don't do debug builds
Previously this recipe was changed to do debug builds because otherwise insane warns that the binary is already stripped. However, debug builds for boost.build also pass -O0. It turns out that given how large Boost is (or, how bad boost.build is) doing a release build with -O3 knocks a third off the walltime for a Boost package in my test, mainly by reducing how long it spends deciding that nothing needs to be rebuilt in do_install: PKG TASK ABSDIFF RELDIFF WALLTIME1 -> WALLTIME2 boost do_install -330.7s -69.2% 477.6s -> 146.9s boost do_compile -7.1s -2.7% 269.3s -> 262.2s Replace debug mode with INSANE_SKIP=already-stripped. (From OE-Core rev: 66d583d1b2bc54cac278c30b5dbc9fde016eb6ee) Signed-off-by: Ross Burton <ross.burton@arm.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/recipes-support/boost/files/0001-build.sh-use-DNDEBUG-also-in-debug-builds.patch')
-rw-r--r--meta/recipes-support/boost/files/0001-build.sh-use-DNDEBUG-also-in-debug-builds.patch27
1 files changed, 0 insertions, 27 deletions
diff --git a/meta/recipes-support/boost/files/0001-build.sh-use-DNDEBUG-also-in-debug-builds.patch b/meta/recipes-support/boost/files/0001-build.sh-use-DNDEBUG-also-in-debug-builds.patch
deleted file mode 100644
index 4c6ef2ed84..0000000000
--- a/meta/recipes-support/boost/files/0001-build.sh-use-DNDEBUG-also-in-debug-builds.patch
+++ /dev/null
@@ -1,27 +0,0 @@
1From 2afd025997a57794ce24e07e914b461dfea6ba5f Mon Sep 17 00:00:00 2001
2From: Alexander Kanavin <alex.kanavin@gmail.com>
3Date: Mon, 26 Aug 2019 16:04:16 +0200
4Subject: [PATCH] build.sh: use -DNDEBUG also in debug builds
5
6Without it, there is a significant performance regression
7when running 'bjam install'.
8
9Upstream-Status: Inappropriate [configuration]
10Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
11---
12 tools/build/src/engine/build.sh | 2 +-
13 1 file changed, 1 insertion(+), 1 deletion(-)
14
15diff --git a/tools/build/src/engine/build.sh b/tools/build/src/engine/build.sh
16index a1e4cd335..64e0a4c80 100755
17--- a/tools/build/src/engine/build.sh
18+++ b/tools/build/src/engine/build.sh
19@@ -436,7 +436,7 @@ case $B2_OS in
20 ;;
21 esac
22
23-if check_debug_build "$@" ; then B2_CXXFLAGS="${B2_CXXFLAGS_DEBUG}"
24+if check_debug_build "$@" ; then B2_CXXFLAGS="${B2_CXXFLAGS_DEBUG} -DNDEBUG"
25 else B2_CXXFLAGS="${B2_CXXFLAGS_RELEASE} -DNDEBUG"
26 fi
27 echo_run ${B2_CXX} ${CXXFLAGS} ${B2_CXXFLAGS} ${B2_SOURCES} -o b2