[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