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

Speedup lexer.currentNoLimit by returning or using field _curToken directly #2943

Merged
merged 6 commits into from
Apr 22, 2024

Conversation

fridis
Copy link
Member

@fridis fridis commented Apr 20, 2024

This is based on PR #2942, only the last commit is added.

First, experimental hack that requires 4x the source file size as extra memory.
Als0 renamed codePointInLine as codePointIndentation.
Speeds up fz for large source files such as when building java.base.fum.
Total speed up of fast line numbers for java.base.fum is now about 10%.
Instead of always searching through all keywords, search only those that have
a suitable length.

This speeds up building java.base.fum by about 5% on my machine.
This speeds up building java.base.fum by about 1% on my machine.
@michaellilltokiwa michaellilltokiwa merged commit eecbf72 into main Apr 22, 2024
5 checks passed
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants