[PATCH, testsuite] Fix PR92464 by adjust test case loop bound

Segher Boessenkool segher@kernel.crashing.org
Wed Nov 13 10:46:00 GMT 2019


Hi!

On Wed, Nov 13, 2019 at 03:31:11PM +0800, Kewen.Lin wrote:
> As PR92464 shows, the recent vectorization cost adjustment on load
> insns is responsible for this regression.  It leads the profitable
> min iteration count to change from 19 to 12.  The case happens to
> hit the threshold.  By actual runtime performance evaluation, the
> vectorized version perform on par with non vectorized version
> (before).  So the vectorization on 12 is actually fine.  To keep
> the case sensitive on high peeling cost, this patch is to adjust
> the loop bound from 16 to 14.

This is fine, okay for trunk, thanks!

Could you add a comment to the testcase saying what it is trying to test
exactly?  It wasn't obvious to me.  Just one or two lines is fine.


Segher



More information about the Gcc-patches mailing list