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.

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

Automate AST generation #680

Open
ForNeVeR opened this issue Oct 29, 2024 · 2 comments
Open

Automate AST generation #680

ForNeVeR opened this issue Oct 29, 2024 · 2 comments
Labels
good-first-issue An issue considered simple enough for new contributors kind:refactor Internal changes not affecting the compiler behavior status:help-wanted Open for contributors

Comments

@ForNeVeR
Copy link
Owner

In #678, we've got a manual AST generator for verification purposes.

For now, it is fully manually written, and I believe this is far from productive.

Let's consider automating it somehow. Some basic ideas:

  1. Get an AST visitor and use it instead of manual enumeration of all nodes
  2. Write a source gen
  3. Convert to JSON and then make some basic string or JSON tree transforms
@ForNeVeR ForNeVeR added status:help-wanted Open for contributors kind:refactor Internal changes not affecting the compiler behavior good-first-issue An issue considered simple enough for new contributors labels Oct 29, 2024
@kant2002
Copy link
Collaborator

I think AST nodes should have information about their range and source file of origin.

@ForNeVeR
Copy link
Owner Author

They should, but this is separate topic I believe. Let's say it's currently anchored by #526.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good-first-issue An issue considered simple enough for new contributors kind:refactor Internal changes not affecting the compiler behavior status:help-wanted Open for contributors
Projects
None yet
Development

No branches or pull requests

2 participants