This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] Fix a problem in lim store motion
- From: Jeffrey A Law <law at redhat dot com>
- To: Zdenek Dvorak <rakdver at atrey dot karlin dot mff dot cuni dot cz>
- Cc: gcc-patches at gcc dot gnu dot org, kenner at vlsi1 dot ultra dot nyu dot edu
- Date: Thu, 29 Jul 2004 09:12:46 -0600
- Subject: Re: [patch] Fix a problem in lim store motion
- Organization: Red Hat, Inc
- References: <20040729131008.GA22272@atrey.karlin.mff.cuni.cz>
- Reply-to: law at redhat dot com
On Thu, 2004-07-29 at 07:10, Zdenek Dvorak wrote:
> Hello,
>
> we were supplying a wrong loop to the set_level function if it was
> called from store motion (when it needed to hoist invariants on that address
> of a memory reference is dependent) -- instead of the loop to that the
> reference belongs we were supplying a loop to that the hoisted statement
> belongs. This caused the ICE during compilation of Ada rts observed by
> Richard Kenner.
>
> Bootstrapped & regtested on i686.
>
> Zdenek
>
> * tree-ssa-loop-im.c (force_move_till_expr, force_move_till):
> Take orig_loop argument and pass it to set_level.
> (schedule_sm): Pass the correct orig_loop to force_move_till.
This is fine. Please install if you haven't done so already.
Thanks,
Jeff