Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Reducing Chains of Single-Child AST Nodes #144

Open
nordlow opened this issue Nov 25, 2014 · 1 comment
Open

Reducing Chains of Single-Child AST Nodes #144

nordlow opened this issue Nov 25, 2014 · 1 comment

Comments

@nordlow
Copy link
Contributor

nordlow commented Nov 25, 2014

As a follow-up to

http://forum.dlang.org/thread/[email protected]#post-dvvrnjipkxixrylkxcki:40forum.dlang.org

I wonder if I have missed something in my usgae of Pegged?

Doesn't Pegged currently waste lots of allocations on chains of single-child node allocations?

Wouldn't it be better to lookup (without involving an AST instance) that, in my case, C.Identifier is an instance of C.DirectDeclarator?
@nordlow nordlow changed the title Reducing Pegged ASTs Reducing Chains of Single-Child AST Nodes Nov 25, 2014
@PhilippeSigaud
Copy link
Collaborator

I answered directly in the D forum thread.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@PhilippeSigaud @nordlow