This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] [RFC] Fix PR 37385 - ICE in set_mem_alias_set with the vectorizer
- From: Ira Rosen <IRAR at il dot ibm dot com>
- To: "Richard Guenther" <richard dot guenther at gmail dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Wed, 10 Sep 2008 10:18:10 +0300
- Subject: Re: [patch] [RFC] Fix PR 37385 - ICE in set_mem_alias_set with the vectorizer
"Richard Guenther" <richard.guenther@gmail.com> wrote on 09/09/2008
12:14:53:
> On Tue, Sep 9, 2008 at 10:20 AM, Ira Rosen <IRAR@il.ibm.com> wrote:
> >
> > The fix of PR 36493
> > (http://gcc.gnu.org/ml/gcc-patches/2008-06/msg01023.html) verifies that
the
> > accesses through the vector pointer and the original scalar memory
> > reference alias. In addition before that patch, the type of vector
store
> > statement was determined by the RHS of the scalar statement, and now it
is
> > determined by the type of LHS, which seems to cause the ICE in
> > set_mem_alias_set, at emit-rtl.c:1789.
> >
> > The attached patch reverts the second part of the PR 36493 patch. This
> > fixes the problem (without hurting PR 36493). I am not sure that the
fix is
> > correct, since the types of RHS and LHS are compatible anyway.
> > I am bootstrapping and testing the patch on x86_64-linux now.
>
> I think it's reasonable. This may be papering over a real problem
somewhere
> else as we are not actually verifying if the types in our statements
> are correct.
Committed revision 140195.
Ira