Rollup of 11 pull requests - #160830
Conversation
When registering selector paths/aliases, steps shouldn't need to know their kind.
Remapping of "rust-analyzer-proc-macro-srv" can be avoided by registering it as an alias attached to the real path. Remapping of "test" is not necessary at all, since `./x test tests` naturally selects all of the test suites in `tests`.
These had a bunch of issues: - They're extra work every time someone opens a PR - They didn't show up at all if people used `gh pr create` - They had "bad vibes" -- reviewers often don't want to think about LLMs, and adding a checkbox makes them very "in-your-face" for the reviewer. - Every PR now shows "1 of 2 tasks", which is useless noise. - The hovered PR description is now useless. Replace them with an HTML comment that says "remember to disclose if you used an LLM". This seems ok and low-noise for now. If we find that people are ignoring the comment, we could edit the triagebot welcome message to include a reminder and a link to the policy.
This fixes RUSTSEC-2026-0233, RUSTSEC-2026-0234 and RUSTSEC-2026-0235.
Many <*const T> methods (read family, copy_to{,_nonoverlapping}, offset_from,
len, as_ptr, get_unchecked), the ptr::{read,write,replace,swap,...} free
functions, and the PartialEq/PartialOrd impls were left as inline while their
<*mut T> equivalents are inline(always). This left them with out-of-line calls
in debug codegen; make them consistent.
This stabilizes both functions under this feature gate: * `std::fs::set_times` * `std::fs::set_times_nofollow`
bootstrap: Remove `PATH_REMAP` from command-line selector handling Remapping of "rust-analyzer-proc-macro-srv" can be avoided by registering it as an alias attached to the real path. Remapping of "test" is not necessary at all, since `./x test tests` naturally selects all of the test suites in `tests`. (This changes the order of steps listed in the relevant snapshot tests, but the set of steps is unchanged.) --- The removal of `ShouldRun::kind` is not essential to this PR, but I included it because it would have conflicted with changes to ShouldRun's alias-related methods.
… r=Mark-Simulacrum Update rustc crate rkyv to 0.8.18 See https://rustsec.org/advisories/RUSTSEC-2026-0233 https://rustsec.org/advisories/RUSTSEC-2026-0234 https://rustsec.org/advisories/RUSTSEC-2026-0235
…hanglo,ShoyuVanilla Clarify `--remap-path-scope` impact on `rustc` metadata This PR clarifirs the impact of the `--remap-path-scope` flag on `rustc` metadata. Fixes rust-lang#159621 r? compiler cc @weihanglo
…rk-Simulacrum Add nightly-only support for Cargo unremap trim-paths files in `rust-gdb` This PR adds support in `rust-gdb` for the un-remapping file produced by Cargo with `-Ztrim-paths` (rust-lang/cargo#17303). This support is nightly-only, and only activated when `RUST_GDB_TRIM_PATHS=unstable` is set. An example of the unremap file: ```json {"v":1} {"rust_version":"1.96.0-nightly","workspace_root":"/home/me/app"} {"from":"/cargo/build-dir","to":"/home/me/app/target"} {"from":"/cargo/registry/6f17d22d3f0a95d1","to":"/home/me/.cargo/registry/src/index.crates.io-6f17d22d3f0a95d1"} {"from":"/rustc/abc123","to":"/home/me/.rustup/toolchains/nightly/lib/rustlib/src/rust"} ```
…e, r=adwinwhite normalization rework: clean up projection_ty_core in rust-lang#160443 I was overly conservative in reworking how normalization works in projection_ty_core, because the PR was about other things. However, we can assume PlaceTy::ty is normalized (... I think). r? lcnr
…acrum Get rid of LLM disclosure checkboxes These had a bunch of issues: - They're extra work every time someone opens a PR - They didn't show up at all if people used `gh pr create` - They had "bad vibes" -- reviewers often don't want to think about LLMs, and adding a checkbox makes them very "in-your-face" for the reviewer. - Every PR now shows "1 of 2 tasks", which is useless noise. - The hovered PR description is now useless. Replace them with an HTML comment that says "remember to disclose if you used an LLM". This seems ok and low-noise for now. If we find that people are ignoring the comment, we could edit the triagebot welcome message to include a reminder and a link to the policy (cc @Kobzol, i believe you'd planned to do this already).
…et-parser, r=clarfonthey Change .expect message on net/parser to follow precondition style
…s, r=tgross35 `extern "custom"`: add tests tracking issue: rust-lang#140829 r? tgross35
…60809, r=nia-e Mark const ptr methods and free functions as inline(always) to match *mut Closes rust-lang#160809. The `*const` read/read_volatile/read_unaligned methods were `#[inline]` while the `*mut` equivalents are `#[inline(always)]` and verified the `*const` forms now inline in debug instead of emitting an out of line call.
…s, r=nia-e Stabilize fs_set_times This stabilizes both functions under this feature gate: * `std::fs::set_times` * `std::fs::set_times_nofollow` This closes rust-lang#147455, which has a completed FCP for this: rust-lang#147455 (comment)
…t-parsing-fn, r=jieyouxu Rename parse_delimited_token_tree in cfg_select <!-- homu-ignore:start --> - [x] I did not use an LLM to create a change in this PR. - [ ] I used an LLM to create a change in this PR, and I have explained below how it was used. From rust-lang#160303 (comment) The fn `parse_delimited_token_tree` only for parsing `cfg_select`.
This comment has been minimized.
This comment has been minimized.
Rollup of 11 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.
What is this?This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.Comparing 811367e (parent) -> 8a2fbe3 (this PR) Test differencesShow 306 test diffsStage 0
Additionally, 298 doctest diffs were found. These are ignored, as they are noisy. Job group index
Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard 8a2fbe3ea881ec68b9f06510fd1a1484cdf5bb6b --output-dir test-dashboardAnd then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
|
📌 Perf builds for each rolled up PR:
previous master: 811367e98c In the case of a perf regression, run the following command for each PR you suspect might be the cause: |
|
Finished benchmarking commit (8a2fbe3): comparison URL. Overall result: ✅ improvements - no action needed@rustbot label: -perf-regression Instruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
Max RSS (memory usage)Results (primary 0.5%, secondary 1.3%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary 0.2%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (primary 0.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 456.009s -> 454.914s (-0.24%) |
Successful merges:
PATH_REMAPfrom command-line selector handling #160675 (bootstrap: RemovePATH_REMAPfrom command-line selector handling )--remap-path-scopeimpact onrustcmetadata #159690 (Clarify--remap-path-scopeimpact onrustcmetadata)rust-gdb#160560 (Add nightly-only support for Cargo unremap trim-paths files inrust-gdb)extern "custom": add tests #160805 (extern "custom": add tests)r? @ghost
Create a similar rollup