S/390: Perform more aggressive inlining

Message ID d248ebfd-063a-97ff-38b4-f124c11d996b@linux.ibm.com
State New
Headers show
Series
  • S/390: Perform more aggressive inlining
Related show

Commit Message

Robin Dapp March 12, 2019, 10:22 a.m.
Hi,

this patch sets the inlining parameters for z13 and later to rather
aggressive values in response to PR85103 that caused performance
regressions in SPEC2006's sjeng and gobmk benchmarks.

Regards
 Robin

--

gcc/ChangeLog:

2019-03-12  Robin Dapp  <rdapp@linux.ibm.com>

        * config/s390/s390.c (s390_option_override_internal): Use more
        aggressive inlining parameters.

Comments

Andreas Krebbel March 12, 2019, 12:11 p.m. | #1
On 12.03.19 11:22, Robin Dapp wrote:
> Hi,

> 

> this patch sets the inlining parameters for z13 and later to rather

> aggressive values in response to PR85103 that caused performance

> regressions in SPEC2006's sjeng and gobmk benchmarks.

> 

> Regards

>  Robin

> 

> --

> 

> gcc/ChangeLog:

> 

> 2019-03-12  Robin Dapp  <rdapp@linux.ibm.com>

> 

>         * config/s390/s390.c (s390_option_override_internal): Use more

>         aggressive inlining parameters.

> 


Ok. Thanks!

Andreas

Patch

diff --git a/gcc/config/s390/s390.c b/gcc/config/s390/s390.c
index aff271806b7..1e268d97ca3 100644
--- a/gcc/config/s390/s390.c
+++ b/gcc/config/s390/s390.c
@@ -14993,6 +14993,18 @@  s390_option_override_internal (struct gcc_options *opts,
 			 opts->x_param_values,
 			 opts_set->x_param_values);
 
+  /* Use aggressive inlining parameters.  */
+  if (opts->x_s390_tune >= PROCESSOR_2964_Z13)
+    {
+      maybe_set_param_value (PARAM_INLINE_MIN_SPEEDUP, 2,
+			     opts->x_param_values,
+			     opts_set->x_param_values);
+
+      maybe_set_param_value (PARAM_MAX_INLINE_INSNS_AUTO, 80,
+			     opts->x_param_values,
+			     opts_set->x_param_values);
+    }
+
   /* Set the default alignment.  */
   s390_default_align (opts);