Skip to content

internal: Bump salsa#22637

Merged
Veykril merged 1 commit into
rust-lang:masterfrom
Veykril:veykril/push-toqyqmnnwvzy
Jun 25, 2026
Merged

internal: Bump salsa#22637
Veykril merged 1 commit into
rust-lang:masterfrom
Veykril:veykril/push-toqyqmnnwvzy

Conversation

@Veykril

@Veykril Veykril commented Jun 25, 2026

Copy link
Copy Markdown
Member

Lots of improvements have landed in this patch release

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 25, 2026
@Veykril Veykril enabled auto-merge June 25, 2026 06:14
@Veykril Veykril added this pull request to the merge queue Jun 25, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 25, 2026
@Veykril Veykril force-pushed the veykril/push-toqyqmnnwvzy branch from 44f0813 to 47cd05b Compare June 25, 2026 06:58
@Veykril Veykril enabled auto-merge June 25, 2026 06:58
@Veykril Veykril added this pull request to the merge queue Jun 25, 2026
Merged via the queue into rust-lang:master with commit cdab17b Jun 25, 2026
18 checks passed
@Veykril Veykril deleted the veykril/push-toqyqmnnwvzy branch June 25, 2026 07:15
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 25, 2026
@Veykril

Veykril commented Jun 25, 2026

Copy link
Copy Markdown
Member Author

cc @MichaReiser, this cut down r-a's memory usage on itself by 70mb! 1501 -> 1435

@MichaReiser

Copy link
Copy Markdown

Nice! You might also see large wins from marking some inputs as NEVER_CHANGE. The impact on ty was minimal (1%), but that's only because most of our queries read some setting which has a lower durability. You could see much better results if you have queries that only read NEVER_CHANGE inputs.

@Veykril

Veykril commented Jun 25, 2026

Copy link
Copy Markdown
Member Author

Yea I was planning to in #22638 but somethings up with CI there, haven't gotten around looking into that.

@MichaReiser

Copy link
Copy Markdown

ohoh :D

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.

3 participants