[PATCH, testsuite] Fix PR92464 by adjust test case loop bound
Kewen.Lin
linkw@linux.ibm.com
Thu Nov 14 06:45:00 GMT 2019
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
More information about the Gcc-patches
mailing list