[gjs/wip/ptomato/mozjs38: 9/28] build: Allow more compiler inlining
- From: Philip Chimento <pchimento src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gjs/wip/ptomato/mozjs38: 9/28] build: Allow more compiler inlining
- Date: Wed, 8 Feb 2017 02:07:36 +0000 (UTC)
commit 231472aa53ceb0e48d952be354c4f3f95fbf070c
Author: Philip Chimento <philip endlessm com>
Date: Tue Feb 7 15:25:42 2017 -0800
build: Allow more compiler inlining
jsapi-util-args.h relies on inlining a lot of functions, into functions
that are relatively small and therefore grow a lot. This started to cause
warnings when switching to SpiderMonkey 38 (presumably because more
methods in SpiderMonkey headers are inlined as well.)
The default value for inline-unit-growth is 30, which according to the
GCC documentation allows functions to grow by 30% as a result of
inlining. Presumably this empirically chosen value of 50 will allow them
to grow by 50%.
https://bugzilla.gnome.org/show_bug.cgi?id=776966
configure.ac | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
---
diff --git a/configure.ac b/configure.ac
index 016118d..d9b942a 100644
--- a/configure.ac
+++ b/configure.ac
@@ -217,6 +217,8 @@ AS_IF([test "x$with_dbus_tests" != "xno"], [
])
AM_CONDITIONAL([DBUS_TESTS], [test "x$with_dbus_tests" != "xno"])
+AX_APPEND_COMPILE_FLAGS(['--param inline-unit-growth=50'])
+
AC_SUBST([gjsjsdir], [\${datadir}/gjs-1.0])
dnl automake 1.11/1.12 defines this but does not substitute it
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]