Re: C++ PATCH: PR 21687

[Mark Mitchell <mark at codesourcery dot com>]

Daniel Berlin wrote:
> On Fri, 2005-09-02 at 11:52 -0700, Mark Mitchell wrote:
>
> > Daniel Berlin wrote:
> >
> > > On Fri, 2005-09-02 at 11:31 -0700, Mark Mitchell wrote:
> > >
> > >
> > > > We were missing ggc_push_context calls around functions in local
> > > > classes; we want to collect within those functions, but we have local
> > > > state that is not in GC roots.
> >
> > > Is there some reason we've chosen not to do the same here?
> >
> > Because we don't know what the local state is.  We're deep in the midst 
> > of parsing one function, and now enter another. 
>
>
> Huh?
> If it's not your local state, something else should be marking it.

I've got an entire parser call-stack, and I don't even know what the 
shape of that call stack is, except that there's a local class at the 
bottom, and a function definition somewhere above.  There may be local 
variables all the way up the stack.

--
Mark Mitchell
CodeSourcery, LLC
mark@codesourcery.com
(916) 791-8304