[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