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

Enhance textDocument/references to Utilize Multi-Core CPUs for Better Performance #17753

Open
eval-exec opened this issue Jul 31, 2024 · 1 comment
Open

Enhance textDocument/references to Utilize Multi-Core CPUs for Better Performance #17753

eval-exec opened this issue Jul 31, 2024 · 1 comment
Labels
C-feature Category: feature request

Comments

@eval-exec
Copy link

eval-exec commented Jul 31, 2024 •
edited
Loading

When invoking the textDocument/references action, the current implementation seems processes references sequentially on a single core (I observed this through htop.). This approach is not utilizing the full potential of modern multi-core processors and leads to increased time cost for finding references in large codebases.

image

I propose enhancing the textDocument/references feature to utilize multi-core CPUs by parallelizing the reference search. Leveraging parallel computing (e.g., using crates like rayon) could significantly reduce the time needed for this operation.

I haven't deeply investigated the rust-analyzer codebase yet; is it possible for this feature to be implemented?

Thank you!
@eval-exec eval-exec added the C-feature Category: feature request label Jul 31, 2024
@davidbarsky
Copy link
Contributor

yes, there is work being on this: #17491

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-feature Category: feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@davidbarsky @eval-exec