[patch] Split Parse Timevar

Lawrence Crowl crowl@google.com
Thu May 27 18:40:00 GMT 2010


On 5/26/10, Jason Merrill <jason@redhat.com> wrote:
> On 04/22/2010 09:15 PM, Lawrence Crowl wrote:
> > The TV_PARSE timevar accounted for too much time to be genuinely
> > useful.  This patch split it into five timevars,
> >
> >     TV_PARSE_ENUM   - time parsing enum definitions
> >     TV_PARSE_FUNC   - time parsing function definitions
> >     TV_PARSE_INMETH - time parsing inline method definitions
> >     TV_PARSE_STRUCT - time parsing struct/union/class definitions
> >     TV_PARSE_GLOBAL - all other time formerly under TV_PARSE
> >
> > Okay for trunk?
>
> Well, as long as we're splitting this up, we should definitely
> have a separate timevar for template instantiation.  And maybe
> one for overload resolution.

That extension wasn't necessary for my original purpose, but it seems
helpful.  Should it be an additional patch or a change to this one?

-- 
Lawrence Crowl



More information about the Gcc-patches mailing list