[Bug testsuite/52614] [4.8 Regression] Test failures in gcc.dg/vect: vectorizing unaligned access

[dominiq at lps dot ens.fr]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52614

--- Comment #7 from Dominique d'Humieres <dominiq at lps dot ens.fr> 2012-03-21 
12:57:54 UTC ---
I have posted the results for powerpc-apple-darwin9 with the patch in comment
#0 at http://gcc.gnu.org/ml/gcc-testresults/2012-03/msg02399.html . It shows
some remaining failures

FAIL: gcc.dg/vect/costmodel/ppc/costmodel-vect-outer-fir.c scan-tree-dump-times
vect "OUTER LOOP VECTORIZED" 2
FAIL: gcc.dg/vect/costmodel/ppc/costmodel-bb-slp-9a.c scan-tree-dump-times slp
"basic block vectorized using SLP" 1

They are fixed with the following patch

---
/opt/gcc/_gcc_clean/gcc/testsuite/gcc.dg/vect/costmodel/ppc/ppc-costmodel-vect.exp
   2011-01-05 08:39:45.000000000 +0100
+++
/opt/gcc/work/gcc/testsuite/gcc.dg/vect/costmodel/ppc/ppc-costmodel-vect.exp   
2012-03-21 10:55:27.000000000 +0100
@@ -34,7 +34,7 @@ if ![is-effective-target powerpc_altivec
 set DEFAULT_VECTCFLAGS ""

 # These flags are used for all targets.
-lappend DEFAULT_VECTCFLAGS "-O2" "-ftree-vectorize" "-fvect-cost-model"
+lappend DEFAULT_VECTCFLAGS "-O2" "-ftree-vectorize" "-fvect-cost-model"
"-fno-common"

 # If the target system supports vector instructions, the default action
 # for a test is 'run', otherwise it's 'compile'.  Save current default.

Note that I don't have write access to SVN, so someone will have to commit the
patches when approved.

Now I wonder if  '-fno-common' should not be the default when
'-ftree-vectorize' is used on the affected targets?