This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] Fix PR/18179 and use get_inner_reference in vectorizer: part 4
- From: Richard Henderson <rth at redhat dot com>
- To: Eric Botcazou <ebotcazou at libertysurf dot fr>
- Cc: Andreas Jaeger <aj at suse dot de>, gcc-patches at gcc dot gnu dot org, Ira Rosen <IRAR at il dot ibm dot com>, Dorit Naishlos <DORIT at il dot ibm dot com>
- Date: Sat, 1 Jan 2005 12:34:14 -0800
- Subject: Re: [patch] Fix PR/18179 and use get_inner_reference in vectorizer: part 4
- References: <20041223205912.GG19547@redhat.com> <20041228195345.GC7332@redhat.com> <m3is6kvxnd.fsf@gromit.moeb> <200501011833.43567.ebotcazou@libertysurf.fr>
On Sat, Jan 01, 2005 at 06:33:43PM +0100, Eric Botcazou wrote:
> loop at vect-65.c:37: not vectorized: unsupported unaligned load.
> loop at vect-65.c:46: not vectorized: bad loop form. multiple exits.
> loop at vect-65.c:57: not vectorized: unsupported unaligned load.
> loop at vect-65.c:66: not vectorized: bad loop form. multiple exits.
>
> So it looks like we now need unaligned loads to vectorize the loops in the
> testcase. Is this expected?
Nope, something broke. We should have been able to forcibly align
this case.
Vectorization folk, it looks like we might need to modify the testsuite
a bit so that we can notice when this sort of thing happens. Either a
--param value that disables unaligned vectors, or better pattern matching
in scan-tree-dump-times that notices when we used misalignment when we
shouldn't have.
r~