[PATCH] See through ADDR_EXPR for forwprop

Richard Guenther rguenth@tat.physik.uni-tuebingen.de
Fri May 27 13:37:00 GMT 2005


Jeffrey A Law wrote:
> On Thu, 2005-05-26 at 16:39 +0200, Richard Guenther wrote:
> 
>> 
>>
>>>I'm pretty sure you can't have an ADDR_EXPR on the LHS, so I suspect
>>>checking for it is just a tiny waste of time.  I'm going to rebootstrap
>>>and retest with the first hunk removed.
>>
>>Ok, I only had this for completeness.  I'll need to learn more of what
>>exactly is gimple and what not.
> 
> It's more a question of what makes sense vs what is gimple in this
> case :-)  You can't change the address of an object, which is in
> effect what you'd be trying to do if an ADDR_EXPR as on the LHS of
> an assignment.

Of course - I thought of sth like

 &(&D.555->foo)->bar->a = ...

and us replacing D.555 - but this would have been folded earlier I
guess.  This is also what I was not sure would be gimple or not.

> Anyway, I've bootstrapped and re-run the testsuite with just the
> change to "see through" ADDR_EXPRs in the RHS of the use statement.
> All is well and good.
> 
> I've attached the actual patch I installed.

Thanks,
Richard.



More information about the Gcc-patches mailing list