This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Load PRE for Array References
- From: Daniel Berlin <dberlin at dberlin dot org>
- To: Andrew Pinski <pinskia at physics dot uc dot edu>
- Cc: Gcc Patch List <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 17 Mar 2006 13:08:11 -0500
- Subject: Re: [PATCH] Load PRE for Array References
- References: <2b8ce7134ab8cf81f9a3d9119af9e408@physics.uc.edu>
On Fri, 2006-03-17 at 11:10 -0500, Andrew Pinski wrote:
> While Looking PR 21485 to figure out a way to fix the code, I changed
> the code
> to use array references. And then I noticed load PRE on the tree level
> was not
> happening for them. So I decided to implement it since the
> infrastructure was all
> there and Daniel Berlin said he was just lazy to implement it.
>
> OK? Bootstrapped and tested on x86_64-linux-gnu twice and
> powerpc-darwin three times
> without any regressions.
>
> Two of the new testcases are xfailed, one I did not look into why it
> failed, the other
> fails because we have:
> b[a_1]
> if (a_1)
> a_2 = a_1+1;
> a_3 = PHI <0, a_2>
> b[a_3]
> and PRE does not recognize that 0 is the same as a_1, I don't know how
> to fix
> this and it might be too hard to fix in general.
Errr, what do you mean "0 is the same as a_1".?