Hi Segher, on 2019/11/13 下午6:42, Segher Boessenkool wrote: > 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. >
Thanks! Done with additional comments, committed in r278195. BR, Kewen
