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

Line-by-line parsing #815

Open
evhub opened this issue Dec 9, 2023 · 1 comment
Open

Line-by-line parsing #815

evhub opened this issue Dec 9, 2023 · 1 comment
Labels
partially resolved performance

Comments

@evhub
Copy link
Owner

evhub commented Dec 9, 2023

If we introduced parsing individually for each lineitem, we could very easily cache and reuse prefix parses by storing the full compiler state after the lineitem. Would require adjusting locs.
@evhub evhub added this to the v3.1.0 milestone Dec 9, 2023
evhub added a commit that referenced this issue Dec 22, 2023
@evhub
Add line-by-line parsing
d6d9e51 
Refs   #815.
@evhub evhub removed this from the v3.1.0 milestone Jan 20, 2024
@evhub
Copy link
Owner Author

evhub commented Jan 20, 2024

Line-by-line parsing is supported now, but it slows things down enough that it doesn't seem worth trying to implement caching for this right now.

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

No branches or pull requests
1 participant
@evhub