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

Rewrite folding range #954

Merged
merged 1 commit into from
Feb 1, 2023

Conversation

Techatrix
Copy link
Member

Just wanted to improve the performance of folding ranges but ended up rewriting the hole thing from scratch, but at least its now slightly faster ( ͡° ͜ʖ ͡°)

I've measured the time to generate folding ranges for Sema.zig.

time per request speedup compared to master 🚀
debug-master 77683ms -
debug-pr 105.2ms 738.4x
safe-master 9584ms -
safe-pr 14.8ms 647.5x
small-master 12819ms -
small-pr 17.8ms 720.1x
fast-master 6176ms -
fast-pr 12ms 515.6x

@SuperAuguste
Copy link
Member

Oh my god Techatrix you're actually cracked 🤯

@SuperAuguste SuperAuguste added the pr:fuzz Attach to a PR to start fuzzing / continually fuzz (please do this before merging!) label Feb 1, 2023
@SuperAuguste SuperAuguste merged commit 1b3c3de into zigtools:master Feb 1, 2023
@Techatrix Techatrix deleted the rewrite-folding-range branch February 7, 2023 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:fuzz Attach to a PR to start fuzzing / continually fuzz (please do this before merging!)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants