Skip to content

Minor query cleanups#153386

Merged
rust-bors[bot] merged 2 commits intorust-lang:mainfrom
nnethercote:minor-query-cleanups
Mar 5, 2026
Merged

Minor query cleanups#153386
rust-bors[bot] merged 2 commits intorust-lang:mainfrom
nnethercote:minor-query-cleanups

Conversation

@nnethercote
Copy link
Contributor

Details in individual commits.

r? @petrochenkov

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Mar 4, 2026
@rust-log-analyzer

This comment has been minimized.

@nnethercote
Copy link
Contributor Author

Looks like a problem with the first commit. I will investigate.

@petrochenkov petrochenkov added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 4, 2026
@nnethercote
Copy link
Contributor Author

The tests/incremental/rustc-rust-log.rs test runs successfully for me locally. Hmm.

It has no effect. Presumably at some point in the past there were
generics involved here?
@nnethercote nnethercote force-pushed the minor-query-cleanups branch 2 times, most recently from 2000f3c to 69042d1 Compare March 4, 2026 23:30
@nnethercote
Copy link
Contributor Author

Down to two commits! Hopefully these ones are ok.

@rustbot ready

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Mar 4, 2026
@petrochenkov
Copy link
Contributor

@bors r+ rollup

@rust-bors
Copy link
Contributor

rust-bors bot commented Mar 5, 2026

📌 Commit 69042d1 has been approved by petrochenkov

It is now in the queue for this repository.

@rust-bors rust-bors bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 5, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Mar 5, 2026
…r=petrochenkov

Minor query cleanups

Details in individual commits.

r? @petrochenkov
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Mar 5, 2026
…r=petrochenkov

Minor query cleanups

Details in individual commits.

r? @petrochenkov
rust-bors bot pushed a commit that referenced this pull request Mar 5, 2026
…uwer

Rollup of 9 pull requests

Successful merges:

 - #153399 (constify `Vec::{into, from}_raw_parts{_in|_alloc}`)
 - #153436 (Use shlex instead of shell-words)
 - #150845 (bootstrap: minimal fix for ./x install src with build.docs = false)
 - #152906 (Make `const_lit_matches_ty` check literal suffixes for exact type match)
 - #153378 (Rename `QueryCache::iter` to `for_each`)
 - #153386 (Minor query cleanups)
 - #153422 (Add a comment explaining the 'tcx lifetime.)
 - #153435 (Fix obtaining def_id from unresolved segment)
 - #153453 (Fix ICEs due to incomplete typechecking on struct literals with syntax errors.)
@rust-bors rust-bors bot merged commit 1e70647 into rust-lang:main Mar 5, 2026
11 checks passed
@rustbot rustbot added this to the 1.96.0 milestone Mar 5, 2026
rust-timer added a commit that referenced this pull request Mar 5, 2026
Rollup merge of #153386 - nnethercote:minor-query-cleanups, r=petrochenkov

Minor query cleanups

Details in individual commits.

r? @petrochenkov
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants