Re: push parser implemenation

[Bob Rossi]

On Wed, Apr 19, 2006 at 09:26:50PM -0400, Joel E. Denny wrote:
> On Mon, 17 Apr 2006, Bob Rossi wrote:
> 
> > Hi everyone, I hope I can get some suggestions soon. I don't want this
> > to be forgotten about. 
> > 
> > Either we can make the push parser a separate file
> 
> There are 4 skeletons now.  Push parser mode could be desirable for any of 
> them.  Coding the push parser in separate skeletons could mean a total of 
> 8 skeletons and a lot of redundant code.  This doesn't seem like the right 
> way to go.

I agree. Plus, the changes are not large.

> >, we can allow the user to loose the 10% speed performance
> 
> I'm not concerned with this performance loss much for my current usage, 
> but I imagine this is not acceptable to many Bison users.

I have a patch that I'll post here shortly. Still trying to generate a
parser that works. It should increase the performance and get yacc mode
within 1% of it's original speed.

> >, I can attempt to do the above
> > suggestion, or
> > 
> > I can see what happens if I declare the struct globally, and when in
> > push-yacc and pure-yacc mode, declare variables on the stack with the
> > same name as the struct's fields. This may improve the speed.
> 
> I don't have time to study these two alternatives and form a thoughtful 
> opinion.  It seems that others don't at the moment either.  Maybe you 
> could try one or both and report your findings.

I appreciate the honest suggestion. After I post the next patch, maybe I
could get some feedback. It is a smaller patch.

Bob Rossi