[gjs] build: Allow more compiler inlining



commit 62a998a15c3ced6e0a1b0102271c8e46b2062f44
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]