Skip to content

internal: LRU body_with_source_map query #17670

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

Merged
merged 1 commit into from
Jul 22, 2024
Merged

internal: LRU body_with_source_map query #17670

merged 1 commit into from
Jul 22, 2024

Conversation

Veykril
Copy link
Member

@Veykril Veykril commented Jul 22, 2024

This query is being invalidated all the time anyways (we have an extra query on top of it for the body incrementality that is not source dependent), so there is little reason to keep these around all the time when only some IDE features are interested in them.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 22, 2024
@Veykril
Copy link
Member Author

Veykril commented Jul 22, 2024

@bors r+

@bors
Copy link
Contributor

bors commented Jul 22, 2024

📌 Commit bd359b3 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Jul 22, 2024

⌛ Testing commit bd359b3 with merge 36f58f5...

@bors
Copy link
Contributor

bors commented Jul 22, 2024

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing 36f58f5 to master...

@bors bors merged commit 36f58f5 into rust-lang:master Jul 22, 2024
11 checks passed
@lnicola lnicola changed the title LRU body_with_source_map query internal: LRU body_with_source_map query Jul 22, 2024
@Veykril Veykril deleted the mem branch July 22, 2024 15:12
@lnicola
Copy link
Member

lnicola commented Jul 22, 2024

This is a 61 MB improvement on self.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants