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, Segher
