-
Notifications
You must be signed in to change notification settings - Fork 26
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.
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
Refactor the compiler to use visitors #75
Comments
Just made a repo with some visitor stuff from the Zero-Overhead blog post: brendanzab/rust-visitors. |
I also have a (currently broken) branch with some experiments at https://github.com/pikelet-lang/pikelet/tree/visitors |
One concern is that Moniker (the name-binding library we use) is currently designed in a way that might make this hard to get much value out of. It would be neat to see if we could move Moniker to a model that more looks like what is described in Visitors Unchained and implemented in alphaLib. |
At the moment we create lots of intermediate data structures between the stages in the compiler, and it would be nice to cut that down!
Using the visitor pattern might make things more efficient by allowing us to fuse various traversals into single passes. Ultimately it might even be a good idea (or even necessary) to push the visitor abstraction back to the nameless library as well. The challenge would be to continue to maintain a nice, readable codebase in the process.
Resources
libsyntax/visit.rs
(visitor module in Rustc) - uses 'walkers' to allow for default methods to simplify visitor implementations.ratel-visitor
- has an interestingbuild!
macroSerializer
anfDeserializer
traits follow this pattern.The text was updated successfully, but these errors were encountered: