This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, ARM] Misaligned access support for ARM Neon
- From: Julian Brown <julian at codesourcery dot com>
- To: Ira Rosen <IRAR at il dot ibm dot com>
- Cc: gcc-patches at gcc dot gnu dot org, Paul Brook <paul at codesourcery dot com>, Richard Earnshaw <rearnsha at arm dot com>
- Date: Tue, 18 May 2010 16:27:50 +0100
- Subject: Re: [PATCH, ARM] Misaligned access support for ARM Neon
- References: <20091117171931.053faec2@rex.config> <1259593368.6000.40.camel@e200601-lin.cambridge.arm.com> <20091218172204.3c0eab5b@rex.config> <200912211220.12265.paul@codesourcery.com> <20100518013108.5514d986@rex.config> <OF67DC3BE7.95207878-ONC2257727.002D3FBA-C2257727.002FBD73@il.ibm.com>
On Tue, 18 May 2010 11:41:26 +0300
Ira Rosen <IRAR@il.ibm.com> wrote:
> Julian Brown <julian@codesourcery.com> wrote on 18/05/2010 03:31:08
> > With these tweaks, test results for gcc/vect.exp change as follows:
> >
> > New FAIL: default/gcc.sum:gcc.dg/vect/vect-109.c scan-tree-dump-
> > times vect "Vectorizing an unaligned access" 10
>
> I think, you may want to add quad_vectors requirement here as well,
> since the test assumes 4 elements for vector int.
Thanks, adding the quad_vectors option fixes that test.
> > New FAIL: default/gcc.sum:gcc.dg/vect/vect-outer-4c.c scan-tree-
> > dump-times vect "OUTER LOOP VECTORIZED" 1
>
> This one requires multiplication of vector short
I might be able to fix that with a follow-up patch.
Full testing revealed some more new failures:
New FAIL: default/g++.sum:g++.dg/vect/pr36648.cc scan-tree-dump-times vect "vectorized 1 loops" 1
New FAIL: default/g++.sum:g++.dg/vect/pr36648.cc scan-tree-dump-times vect "vectorizing stmts using SLP" 1
These are due to a transformation in an earlier tree pass (091t.crited
I think), leading to:
pr36648.cc:10: note: not vectorized: control flow in loop.
I don't think this indicates a failure in this patch, as such.
Julian