[PATCH] testsuite/rs6000: Add option to ignore vect cost model

Segher Boessenkool segher@kernel.crashing.org
Thu Jul 16 20:31:04 GMT 2020


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


More information about the Gcc-patches mailing list