Rollup of 10 pull requests#159604
Closed
JonathanBrouwer wants to merge 113 commits into
Closed
Conversation
…onstant-sized We used to set a `slice: None`.
The InvocationStrategy::Once path indexed world.flycheck[0] directly. The always!() above only logs in release builds, so when the list was empty it fell through and panicked with index out of bounds. Use first() and skip the restart when there is no handle. Fixes rust-lang/rust-analyzer#21638
Previously we would still try to rename the incr comp session dir after deleting it.
In both cases the incr comp session dir should no longer be accessed.
They should skip non-module scope (although I think they cannot skip and will do just fine), but must also process block module scopes.
Ask for disclosure in AI contributions
internal: update links to `rustc_hir_analysis`
fix: Resolve non-plain paths in blocks correctly
…tpluxqp feat: Treat library files as truly client immutable
… claude lifetime bound scope
…ztynowk internal: Split out a session construct
feat: early late classification of lifetimes
…-from-location, r=nnethercote Enable single Location to issue multiple borrows In the Reborrow work, a single MIR assignment statement can produce multiple borrows. To support that the GatherBorrows contents need to change a little. Aside from the `get_index_of` function which is used in three places, none of the changes affect code outside the `borrow_set.rs` file. I also changed the one `Vec` there was into a `SmallVec<[1]>` as that is the most likely case (I believe), and similarly used `SmallVec<[1]>` for the `Location->BorrowIndex[]` listing. r? compiler
…chenkov Improve `AttrItem::span` It has some shortcomings. Details in individual commits. r? @petrochenkov
…, r=lcnr Do not mark unnormalized const aliases as rigid when normalizing param env Fixes rust-lang/trait-system-refactor-initiative#279 The old solver actually evaluates const aliases before normalizing param env. The next solver should be consistent with that. r? lcnr also cc @BoxyUwU
…s-test, r=nnethercote Add regression test for nested replacement ranges in `collect_tokens` rust-lang#129346 removed support for nested replacement ranges in `collect_tokens`, and simplified the nesting out of `tests/ui/proc-macro/macro-rules-derive-cfg.rs` accordingly. That removal turned out to break real-world code, so it was reverted in rust-lang#132587 — but the revert only restored the compiler code, not the original nested shape of the test, leaving the nested case uncovered ever since. This reinstates the nested case as a separate test: the pre-rust-lang/rust#129346 shape of `macro-rules-derive-cfg.rs` (a `cfg_attr` expansion nested inside a `let` statement, nested inside an anonymous constant), combined with the extra inner attributes that rust-lang#129346 added to the simplified version. If nested replacement range support regresses again, this test fails. The expected stdout was verified against current nightly; the sibling test's checked-in stdout reproduces byte-for-byte under the same setup. Closes rust-lang#132727
…ovements, r=Kobzol Minor `TokenStream` improvements I found these while attempting to improve the `TokenStream` representation in rust-lang#159048. r? @Kobzol
…li-obk Separate `InterpCx` usage by `ConstAnalysis` phases `ConstAnalysis` has a `RefCell<InterpCx>`. During the analysis phase the accesses are read-only, but the visiting phase (performed by `Collector`) requires mutation, because of calls to `try_make_constant`. This commit removes the `RefCell` by having two `InterpCx` instances: a read-only one for the `ConstAnalysis` and a mutable one for the visiting phase, stored in `Collector`. This removes the need for the `OperandCollector::ecx` field, and for many `borrow`/`borrow_mut` calls. r? @cjgillot
…=Kobzol Clarify `push_stream`/`push_tree` More `TokenStream` cleanups. Details in individual commits. r? @Kobzol
Contributor
Author
Contributor
This comment has been minimized.
This comment has been minimized.
rust-bors Bot
pushed a commit
that referenced
this pull request
Jul 20, 2026
Rollup of 10 pull requests try-job: dist-various-1 try-job: test-various try-job: x86_64-gnu-aux try-job: x86_64-gnu-llvm-21-3 try-job: x86_64-msvc-1 try-job: aarch64-apple try-job: x86_64-mingw-1 try-job: i686-msvc-*
This comment has been minimized.
This comment has been minimized.
rust-bors Bot
pushed a commit
that referenced
this pull request
Jul 20, 2026
…uwer Rollup of 10 pull requests Successful merges: - #159600 (`rust-analyzer` subtree update) - #158046 (proc_macro: preserve file module spans for inner attrs) - #159000 (Small cleanups to the incr comp session code) - #159449 (Enable single Location to issue multiple borrows) - #159587 (Improve `AttrItem::span`) - #159460 (Do not mark unnormalized const aliases as rigid when normalizing param env) - #159529 (Add regression test for nested replacement ranges in `collect_tokens`) - #159585 (Minor `TokenStream` improvements) - #159586 (Separate `InterpCx` usage by `ConstAnalysis` phases) - #159603 (Clarify `push_stream`/`push_tree`) Failed merges: - #159590 (Remove some dead code)
Collaborator
|
The job Click to see the possible cause of the failure (guessed by this bot)Important For more information how to resolve CI failures of this job, visit this link. |
Contributor
|
💔 Test for 9169afd failed: CI. Failed job:
|
Contributor
Contributor
|
This pull request was unapproved due to being closed. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Successful merges:
rust-analyzersubtree update #159600 (rust-analyzersubtree update)AttrItem::span#159587 (ImproveAttrItem::span)collect_tokens#159529 (Add regression test for nested replacement ranges incollect_tokens)TokenStreamimprovements #159585 (MinorTokenStreamimprovements)InterpCxusage byConstAnalysisphases #159586 (SeparateInterpCxusage byConstAnalysisphases)push_stream/push_tree#159603 (Clarifypush_stream/push_tree)Failed merges:
r? @ghost
Create a similar rollup