| Message ID | 54802C8B.9010605@arm.com |
|---|---|
| State | New |
| Headers | show |
On 4 December 2014 at 09:42, Kyrill Tkachov <kyrylo.tkachov@arm.com> wrote: > 2014-12-04 Kyrylo Tkachov kyrylo.tkachov@arm.com\ > > * gcc.target/aarch64/vect_ctz_1.c: Add -fno-vect-cost-model to > dg-options. OK /Marcus
Index: gcc/testsuite/gcc.target/aarch64/vect_ctz_1.c =================================================================== --- gcc/testsuite/gcc.target/aarch64/vect_ctz_1.c (revision 218233) +++ gcc/testsuite/gcc.target/aarch64/vect_ctz_1.c (working copy) @@ -1,5 +1,5 @@ /* { dg-do run } */ -/* { dg-options "-O3 -save-temps -fno-inline" } */ +/* { dg-options "-O3 -save-temps -fno-inline -fno-vect-cost-model" } */ extern void abort ();
Hi all, The second scan-assembler test for clz\tv\[0-9\]+\.2s FAILs on this test due to vector costs on A57. The vectorisation happens for cortex-a53 and thunderx. I think this test was supposed to test the capability of vectorising clz rather than the tuning decision of whether to (though I imagine people can argue the other way), so this patch adds an -fno-vect-cost-model to it to get it to always vectorise. This way the test passes on all -mcpu options. Ok for trunk? Thanks, Kyrill 2014-12-04 Kyrylo Tkachov kyrylo.tkachov@arm.com\ * gcc.target/aarch64/vect_ctz_1.c: Add -fno-vect-cost-model to dg-options.