RFA: Fix ICE related to ADDR_EXPR forward propagation

Jeffrey A Law law@redhat.com
Fri May 20 14:44:00 GMT 2005


On Fri, 2005-05-20 at 07:12 -0400, Daniel Berlin wrote:
> On Fri, 2005-05-20 at 07:23 +0100, Richard Sandiford wrote:
> > Mainline currently segfaults while trying to build newlib's hash.c.
> > The problem seems to be caused by the recent patch to forward-propagate
> > ADDR_EXPRs.  The idea of those changes is to replace:
> > 
> >     t1 = &thing[0]
> >     t2 = index * scale
> >     t3 = (ptr_type) t2
> >     result = t1 + t3
> > 
> > with:
> > 
> >     result = &thing[index]
> 
> 
> >   I'm not exactly 100%
> > confident this will be accepted as the right fix though. ;)
> 
> 
> You want to call recompute_tree_invariant_for_addr_expr on the ADDR_EXPR
> after the operands are replaced.
> 
> It'll do what it says (recompute the invariant flag, etc, as
> approriate).
Andrew Pinski has a patch for that -- I've twiddled it slightly
(moved it into the cleanup routine recently added by RTH).

I just need to verify that it still fixes the bugs it's supposed
to fix and it'll be ready to be checked in.

jeff




More information about the Gcc-patches mailing list