Hi, on 2020/7/17 上午4:31, Segher Boessenkool wrote: > 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
Oh my poor naming skills ;-) Will fix it with this. > or even > rs6000/test: Add -fno-vect-cost-model to some tests > > > Okay for trunk. Thanks! Thanks! Kewen
