On Dec 19, 2024, Richard Biener <richard.guent...@gmail.com> wrote:

> Please don't use -1 initializers, instead populate
> opts.cc:default_options_table.

ACK.  This trick (that I used) will be hard to unlearn ;-)

> IMO options where it's not clear how they interact are bad.

*nod*.  I struggled a bit with that.

In the end, I couldn't get any evidence that ifcombine was the culprit
by the slowdowns, so I'm inclined to withdraw the patch and leave it at
that.  We have an option to disable individual passes already.

If we find evidence that limiting the attempts to combine noncontiguous
blocks is needed, I'll be happy to introduce a param for that.

Thanks,

-- 
Alexandre Oliva, happy hacker            https://FSFLA.org/blogs/lxo/
   Free Software Activist                   GNU Toolchain Engineer
More tolerance and less prejudice are key for inclusion and diversity
Excluding neuro-others for not behaving ""normal"" is *not* inclusive

Reply via email to