[PATCH] rs6000: Rename adjust_vectorization_cost

Kewen.Lin linkw@linux.ibm.com
Wed Jul 22 15:53:04 GMT 2020


Hi Segher,

on 2020/7/22 下午4:26, Segher Boessenkool wrote:
> Hi!
> 
> On Wed, Jul 22, 2020 at 09:44:52AM +0800, Kewen.Lin wrote:
>> This trivial patch is to rename adjust_vectorization_cost to 
>> adjust_vect_cost_per_stmt.  Hope it's more meaningful, as well
>> as to avoid the confusion between the possible to be landed
>> function "adjust_vect_cost" and "adjust_vectorization_cost".
>>
>> Even without "adjust_vect_cost", I guess it's still good?
> 
> It is an improvement for sure, so it is okay for trunk of course.  It
> still isn't very clear from the name how this would differ from
> adjust_vect_cost, but that _is_ obviously the more generic name, so
> that is good.
> 

Thanks!  To make adjust_vect_cost more meaningful, I just updated it
to adjust_vect_cost_per_loop in the latest version, hope it gets
a bit better.  :-)

BR,
Kewen


More information about the Gcc-patches mailing list