This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [autovect] [patch] Make store motion use alias oracle
- From: Revital1 Eres <ERES at il dot ibm dot com>
- To: Zdenek Dvorak <rakdver at atrey dot karlin dot mff dot cuni dot cz>
- Cc: Ayal Zaks <ZAKS at il dot ibm dot com>, Dorit Nuzman <DORIT at il dot ibm dot com>, gcc-patches at gcc dot gnu dot org, Victor Kaplansky <VICTORK at il dot ibm dot com>
- Date: Mon, 12 Mar 2007 20:46:03 +0200
- Subject: Re: [autovect] [patch] Make store motion use alias oracle
> > Index: tree-if-conv.c
> > ===================================================================
> > --- tree-if-conv.c (revision 122819)
> > +++ tree-if-conv.c (working copy)
> > @@ -334,9 +334,42 @@
> > return true;
> > }
> >
> > +/* Return true if STMT may have certain side effects
> > + and thus ifcvt can not be applied. Otherwise return false. */
> > +static bool
> > +may_have_side_effect_p (tree stmt)
> > +{
> > + tree lhs, rhs;
> > +
> > + if (flag_unswitch_loops
> > + && TREE_CODE (stmt) == COND_EXPR)
> > + return false;
>
> this condition seems wrong (in ifcvt, you probably should never see
> COND_EXPR here).
I had doubts about that myself but I wanted to stay
compatible with the original function. I will test
the patch without that condition.
Thanks,
Revital
>
> Zdenek