On Wed, May 20, 2020 at 12:30:30PM +0200, Richard Biener wrote:
> I think this is the wrong way to approach this.  You're doing too many
> things at once.  Try to fix the powerpc regression with the extra
> flag_rtl_unroll_loops, that could be backported.  Then you can
> independently see whether enabling more unrolling at -O2 makes
> sense.  Because currently we _do_ unroll at -O2 when it does
> not increase size.  Its just your patches make this as aggressive
> as -O3.

Just do a separate flag (and option) for cunroll, instead?

The RTL unroller is *the* unroller, and has been since forever.


Segher

Reply via email to