Rollup of 12 pull requests#152063
Conversation
This has the nice side-effect of eliminating `rustc_codegen_ssa`'s dependency on `rustc_query_system`. (Indeed, looking through such dependencies was how I found this.)
It probably wasn't committed after an earlier change.
Show largest job duration changes in hours and minutes in the post-merge report This was suggested by... someone, somewhere. But it's a good idea. See rust-lang#152025 (comment) - now instead of seconds it would show this: # Job duration changes 1. [dist-apple-various](https://github.com/rust-lang/rust/actions/runs/21617345733#summary-62299011925): 1h 3m -> 1h 29m (+41.5%) 2. [dist-x86_64-apple](https://github.com/rust-lang/rust/actions/runs/21617345733#summary-62299011972): 2h 1m -> 2h 21m (+16.4%) 3. [dist-ohos-x86_64](https://github.com/rust-lang/rust/actions/runs/21617345733#summary-62299011829): 1h 11m -> 1h 19m (+12.4%) 4. [x86_64-gnu-llvm-20](https://github.com/rust-lang/rust/actions/runs/21617345733#summary-62299011909): 1h 9m -> 1h 17m (+11.9%) 5. [pr-check-1](https://github.com/rust-lang/rust/actions/runs/21617345733#summary-62299011862): 32m 27s -> 28m 44s (-11.4%) 6. [tidy](https://github.com/rust-lang/rust/actions/runs/21617345733#summary-62299011845): 2m 37s -> 2m 52s (+9.9%) 7. [armhf-gnu](https://github.com/rust-lang/rust/actions/runs/21617345733#summary-62299011763): 1h 22m -> 1h 30m (+9.5%) 8. [dist-aarch64-apple](https://github.com/rust-lang/rust/actions/runs/21617345733#summary-62299011941): 1h 54m -> 1h 44m (-9.1%) 9. [x86_64-gnu-llvm-21-3](https://github.com/rust-lang/rust/actions/runs/21617345733#summary-62299011924): 1h 45m -> 1h 54m (+9.1%) 10. [x86_64-gnu-llvm-21-1](https://github.com/rust-lang/rust/actions/runs/21617345733#summary-62299011923): 1h 9m -> 1h 15m (+9.0%)
…ukang coverage: Add a test case for a previously-unknown span mismatch - This is a revised version of rust-lang#152036. --- In rust-lang#145643, a defensive check was added to detect spans that unexpectedly don't match the context of the function body span. There was no known way to trigger that condition, so a `debug_assert!` was added to make violations easier to notice. A way to trigger the condition using nested macro expansions was subsequently found and reported as an ICE (in debug-assertion builds) in rust-lang#147339. Now that we have a concrete example to investigate, we can remove the `debug_assert!` so that there is no ICE in debug-assertion builds. - Fixes rust-lang#147339. r? chenyukang
…t, r=lqd Convert to inline diagnostics in `rustc_infer` For rust-lang#151366
…BoxyUwU Forbid manual `Unpin` impls for structurally pinned types Part of [`pin_ergonomics`](rust-lang#130494). It forbids to `impl Unpin for T` where `T` is an ADT marked with `#[pin_v2]`.
…yUwU Check proj's parent is trait or not when checking dyn compatibility Fixes rust-lang#151708 When checking dyn compatibility, `proj` here may point to free const whose parent is not trait. Then `TraitRef::from_assoc` will call `generics_of` on the wrong parent. After this change, the following case without `#[type_const]` will still emit ICE same to rust-lang#149066, but different to the ICE reported in rust-lang#151708 ```rust #![feature(min_generic_const_args)] // #[type_const] const N: usize = 2; trait CollectArray { fn inner_array(&self) -> [i32; N]; } ``` r? @BoxyUwU
…anBrouwer Port `rustc_mir` to attribute parser Tracking issue: rust-lang#131229
…r=BoxyUwU error on unsized AnonConsts The constant evaluator does not support unsized types, however, unsized AnonConsts were never checked to be Sized, so no errors were generated on them and the constant was attempted to be constant evaluated. This caused the constant evaluator to ICE. Add a special case for AnonConsts in rustc_hir_typeck, as suggested by @BoxyUwU in rust-lang#137582. There is no checking for `#![feature(unsized_const_params)]` which should eventually revert this check when the feature becomes more implemented. That issue is assigned to @el-ev but I started looking into this as a jumping off point / motivation to learn some compiler stuff, and I eventually got to the point of fixing it, so I'm submitting a PR anyway. So just a ping/FYI to @el-ev that I'm submitting this, sorry! There are three relevant github issues to this ICE that I could find: - fixes rust-lang#137582 - fixes rust-lang#151591 The similar issue rust-lang#104685 is NOT fixed, it might be good to glance at that before verifying this particular fix, to make sure this fix is actually in the right place. (I haven't looked at it much) r? @BoxyUwU
Convert to inline diagnostics in `rustc_attr_parsing` Converts a crate for rust-lang#151366 This PR is almost completely autogenerated by a hacky script I have locally :)
…-derives, r=Zalathar `NativeLib` cleanups Improvements to `NativeLib`. r? @Zalathar
…-Simulacrum disable socket tests in Miri rust-lang#150428 added some tests that do not work in Miri since we do not support sockets. r? @Mark-Simulacrum
…rt, r=jdonszelmann Convert to inline diagnostics in `rustc_query_system` For rust-lang#151366 (comment) r? @jdonszelmann (or anyone else who feels like it)
…s, r=GuillaumeGomez Try to fix `rustdoc-gui/globals.goml` flakyness Part of rust-lang#93784. Hopefully this one fixes it... r? ghost
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
@bors retry |
|
💔 Test for 01572a2 failed: CI. Failed job:
|
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 55407b8 (parent) -> 366a1b9 (this PR) Test differencesShow 230 test diffsStage 0
Stage 1
Stage 2
(and 96 additional test diffs) Additionally, 34 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 366a1b93e7f466ebe559477add05f064873d0c71 --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: 55407b8cdb In the case of a perf regression, run the following command for each PR you suspect might be the cause: |
|
Finished benchmarking commit (366a1b9): comparison URL. Overall result: ❌✅ regressions and improvements - please read the text belowOur benchmarks found a performance regression caused by this PR. Next Steps:
@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.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesThis benchmark run did not return any relevant results for this metric. Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 471.991s -> 472.721s (0.15%) |
Successful merges:
rustc_infer#152045 (Convert to inline diagnostics inrustc_infer)Unpinimpls for structurally pinned types #149263 (Forbid manualUnpinimpls for structurally pinned types)rustc_mirto attribute parser #151848 (Portrustc_mirto attribute parser)rustc_attr_parsing#151944 (Convert to inline diagnostics inrustc_attr_parsing)NativeLibcleanups #152019 (NativeLibcleanups)rustc_query_system#152041 (Convert to inline diagnostics inrustc_query_system)rustdoc-gui/globals.gomlflakyness #152050 (Try to fixrustdoc-gui/globals.gomlflakyness)Failed merges:
r? @ghost
Create a similar rollup