diff mbox series

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 | expand

Commit Message

Robin Dapp March 12, 2019, 10:22 a.m. UTC
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. UTC | #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
diff mbox series

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);