Re: Parser Cruft in gram.y - Mailing list pgsql-hackers

From Greg Stark
Subject Re: Parser Cruft in gram.y
Date
Msg-id CAM-w4HM5Pcka3tDTtXACUXNz5qYxAnbbMhq8Fgf1ttR2=yx3og@mail.gmail.com
Whole thread Raw
In response to Re: Parser Cruft in gram.y  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Parser Cruft in gram.y  (Andres Freund <andres@2ndquadrant.com>)
List pgsql-hackers
On Thu, Dec 20, 2012 at 3:18 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Greg Stark <stark@mit.edu> writes:
>> But I'm not entirely convinced any of this is actually useful. Just
>> becuase the transition table is large doesn't mean it's inefficient.
>
> That's a fair point.  However, I've often noticed base_yyparse() showing
> up rather high on profiles --- higher than seemed plausible at the time,
> given that its state-machine implementation is pretty tight.  Now I'm
> wondering whether that isn't coming from cache stalls from trying to
> touch all the requisite parts of the transition table.

For what it's worth the bloat isn't in the parser transition table at all:
516280 yy_transition
147208 yytable
147208 yycheck
146975 base_yyparse
17468 yypact
9571 core_yylex
8734 yystos
8734 yydefact

Unless I'm confused yy_transition is in fact the *lexer* transition
table. I'm not sure how to reconcile that with the profiling results
showing the cache misses in base_yyparse() though.


-- 
greg



pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: operator dependency of commutator and negator, redux
Next
From: Tom Lane
Date:
Subject: Re: [GENERAL] trouble with pg_upgrade 9.0 -> 9.1