Hi! On Thu, Jul 16, 2020 at 02:51:23PM +0800, Kewen.Lin wrote: > In my testing with cost tweaking for vector with length, I found > two cases below didn't get the expected output. Since the expected > instructions reply on the vectorization occurrence, we don't expect > vectorization gets disabled by cost model. > > To make it not fragile, the fix is to force it without vect cost model.
>From the subject it wasn't clear to me that you just add the flag (not implement a new option :-) ), and only to two testcases. Maybe rs6000/test: Add -fno-vect-cost-model to conv-vectorize-[12].c or even rs6000/test: Add -fno-vect-cost-model to some tests Okay for trunk. Thanks! Segher
