This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Kill pointers keeping gimple function body around after RTL is generated
> On Fri, 2004-09-03 at 04:03, Jan Hubicka wrote:
> > > On Thu, Sep 02, 2004 at 11:59:06PM +0200, Jan Hubicka wrote:
> > > > - for (; !bsi_end_p (bsi); bsi_next (&bsi))
> > > > + for (; !bsi_end_p (bsi); bsi_remove (&bsi))
> > >
> > > Why? Aren't we coing to zap the entire statement list in a moment?
> >
> > I am having incremental patch that actually teach bsi_remove to release
> > unshared parts of the bodies as mentioned earlier. (it at least can be
> > made to trivially recycle the containers)
> >
> IIRC, we bsi_remove to move statements around. The fact that we call
> bsi_remove() doesn't necessarily mean we won't need the statement
> anymore.
Yes, I broke it into bsi_remove and bsi_delink and updated offending
uses. (it seemed better than introducing bsi_remove_and_free for the
common case)
Honza
>
>
> Diego.