Re: [patch, testsuite] Fix vectorizer testsuite failures on ARM

[Ira Rosen <IRAR@il.ibm.com>]

Steve Ellcey <sje@cup.hp.com> wrote on 15/06/2011 08:15:27 PM:

>
> > testsuite/ChangeLog:
> >
> >        * gcc.dg/vect/vect-16.c: Rename to ...
> >        * gcc.dg/vect/no-fast-math-vect16.c: ... this.
> >        * gcc.dg/vect/vect-peel-3.c: Adjust misalignment values
> >        for double-word vectors.
> >        * gcc.dg/vect/vect-peel-4.c: Likewise.
> >        * gcc.dg/vect/bb-slp-10.c: Replace vect_hw_misalign with
> >        vect_element_align.
> >        * gcc.dg/vect/vect.exp: Run no-fast-math-* tests with
> >        -fno-fast-math.
>
> Ira,
>
> This change broke vect-peel-3.c and vect-peel-4.c on IA64 HP-UX.
> I get these failures:
>
>    FAIL: gcc.dg/vect/vect-peel-3.c -flto scan-tree-dump-times vect
> "Alignment of access forced using peeling" 1
>    FAIL: gcc.dg/vect/vect-peel-3.c -flto scan-tree-dump-times vect
> "vectorized 1 loops" 1
>    FAIL: gcc.dg/vect/vect-peel-3.c scan-tree-dump-times vect
> "Alignment of access forced using peeling" 1
>    FAIL: gcc.dg/vect/vect-peel-3.c scan-tree-dump-times vect
> "vectorized 1 loops" 1
>    FAIL: gcc.dg/vect/vect-peel-4.c -flto scan-tree-dump-times vect
> "vectorized 1 loops" 1
>    FAIL: gcc.dg/vect/vect-peel-4.c scan-tree-dump-times vect
> "vectorized 1 loops" 1
>
> I think that changing i+6 and i+2 to i+5 and i+1 broke things on HP-UX
> because we are big-endian and the old values may not have worked on ARM
> because it is little-endian (just guessing).
>
> I am not sure what we want to do to make this test work on both
platforms.
> Do you have any ideas?

I don't think it is related to endianess.
Could you please send me the dump files (created with
-fdump-tree-vect-details)?

Thanks,
Ira

>
> Steve Ellcey
> sje@cup.hp.com