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
