[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