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
