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?
to your account
@findleyr profiled didChange and found that this function was taking up the most time in didModifyFiles events, so we should cache the directories on the snapshot.
The text was updated successfully, but these errors were encountered:
This was mentioned in #45686, but a forthcoming benchmark suggests that allKnownSubdirs accounts for more than half of the time it takes to process (but not diagnose) a file change.
Change https://golang.org/cl/317410 mentions this issue: internal/lsp: memoize allKnownSubdirs instead of recomputing
internal/lsp: memoize allKnownSubdirs instead of recomputing
Successfully merging a pull request may close this issue.