On 1/3/20 5:12 PM, Segher Boessenkool wrote:
Hi!

On Thu, Jan 02, 2020 at 12:07:27PM +0100, Martin Liška wrote:
The param is changed here:

   /* Restrict the amount of work combine does at -Og while retaining
      most of its useful transforms.  */
   if (opts->x_optimize_debug)
     SET_OPTION_IF_UNSET (opts, opts_set, param_max_combine_insns, 2);

and so that it should be per-function. The only usage is in combine RTL
pass, which is per-function optimization.

Patch can bootstrap on x86_64-linux-gnu and survives regression tests.

Ready to be installed?

Sure.  Okay for trunk.  Thanks!

Thank you.


2020-01-02  Martin Liska  <mli...@suse.cz>

        PR tree-optimization/92860
        * params.opt: Mart param_max_combine_insns with Optimization
        keyword.

s/mart/mark/

Will fix the typo.


(Btw, does this help -Og at all?  If so, shouldn't it be -O1 as well?)

I would say yes. The param default for -Og was added by Richi in 2014.
I'm adding him to CC.

Martin



Segher


Reply via email to