Repository navigation
Rollup of 6 pull requests - #165361
Rollup of 6 pull requests#165361
Conversation
Use more accurate span when suggesting to remove guard and body of a never-pattern match arm (`feature(never_patterns)`]. ``` error: a guard on a never pattern will never be run --> $DIR/ICE-119271-never-arm-attr-in-guard.rs:8:13 | LL | false | ^^^^^ | help: remove the guard, leaving only the pattern | LL - Some(!) LL - LL - if #[deny(unused_mut)] LL - false LL + Some(!) | ```
… r=fmease Tweak suggestion of "unreachable due to never pattern" Use more accurate span when suggesting to remove guard and body of a never-pattern match arm (`feature(never_patterns)`]. ``` error: a guard on a never pattern will never be run --> $DIR/ICE-119271-never-arm-attr-in-guard.rs:8:13 | LL | false | ^^^^^ | help: remove the guard, leaving only the pattern | LL - Some(!) LL - LL - if #[deny(unused_mut)] LL - false LL + Some(!) | ```
std: fix platform checks for hermit Closes rust-lang#164495. Found this while working on Hermit OS support for Rustix (bytecodealliance/rustix#1672) ```console $ cargo +nightly check -q -Zbuild-std --target x86_64-unknown-hermit error[E0425]: cannot find value `IPV6_UNICAST_HOPS` in crate `c` --> ~/.rustup/toolchains/nightly-aarch64-apple-darwin/lib/rustlib/src/rust/library/std/src/sys/net/connection/socket/mod.rs:534:61 | 534 | setsockopt(&self.inner, c::IPPROTO_IPV6, c::IPV6_UNICAST_HOPS, limit as c_int) | ^^^^^^^^^^^^^^^^^ not found in `c` ... error[E0425]: cannot find value `IPV6_MULTICAST_HOPS` in crate `c` --> ~/.rustup/toolchains/nightly-aarch64-apple-darwin/lib/rustlib/src/rust/library/std/src/sys/net/connection/socket/mod.rs:957:61 | 957 | setsockopt(&self.inner, c::IPPROTO_IPV6, c::IPV6_MULTICAST_HOPS, limit as c_int) | ^^^^^^^^^^^^^^^^^^^ not found in `c` | ``` cc: @Mallets
…oxyUwU create mir for direct-rhs consts Alternative to rust-lang#164982 - see that PR for context. When referring to a direct constant via a `mir::Const::Unevaluated`, synthesize a MIR body, whose contents are simply returning a `mir::Const::Ty` of itself. This effectively provides both a MIR view, as well as a typesystem view, of the same constant, without risking representing the same constant in different ways in the two different representations, because one is just referring to the other. r? @BoxyUwU fyi @oli-obk @lcnr
…eval, r=khyperia test DSL evaluate sequentially For Context: rust-lang/project-assumptions-on-binders#59 changelog: Currently, we made the test DSL evaluate sequentially by adding an ordering `vec`. This `vec` goes through the whole pipeline, including AST -> HIR, HIR transform, HIR -> solver type, and wfcheck. In `wfcheck.rs` and `collect.rs`, everything that revolves around `predicates`, `constraints`, and the like is still almost the same as before. I added a new test file like this, it should fail before this PR and pass after it. ```rust core::test_binder_constraints! { impl<'a: 'b, 'b, 'c: 'd, 'd> { forall<'x> { 'a: 'b, forall<'y> { } expect { 'a: 'b }, 'c: 'd, } } } ``` This test file came up in the earlier discussion. I chose to keep it separate here, which strictly guarantees sequential evaluation and is also simpler to implement :) Happy to make any changes if there are any issues.
Add assembly test for masked index without redundant shifts Fixes rust-lang#56057 This was fixed by the LLVM 18 update in rust-lang#120055
test(lexer): Add test that ZWJ/ZWNJ are accepted in identifiers rust-lang/reference#2400
Rollup of 6 pull requests try-job: dist-various-1 try-job: test-various try-job: test-x86_64-gnu-aux try-job: test-x86_64-msvc-1 try-job: test-aarch64-apple-1 try-job: test-aarch64-apple-2 try-job: test-x86_64-mingw-1 try-job: test-i686-msvc try-job: test-armhf-gnu try-job: test-x86_64-gnu-llvm-22-3 try-job: test-aarch64-msvc-2
This comment has been minimized.
This comment has been minimized.
|
💔 Test for 69872e9 failed: CI. Failed job:
|
|
A job failed! Check out the build log: (web) (plain enhanced) (plain) Click to see the possible cause of the failure (guessed by this bot) |
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 ae36fe2 (parent) -> 1c3a650 (this PR) Test differencesShow 16 test diffsStage 1
Stage 2
Additionally, 7 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 1c3a6506966e21845b31de0b075d2527660db132 --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:
parent commit: ae36fe22c7 In the case of a perf regression, run the following command with the SHAs of each PR you suspect might be the cause: |
|
Finished benchmarking commit (1c3a650): comparison URL. Overall result: ❌✅ regressions and improvements - please read:Our 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 (secondary 2.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary -2.2%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis perf run didn't have relevant results for this metric. Bootstrap: 483.826s -> 483.238s (-0.12%) |
|
@rust-timer triage all |
Running triage with 2 benchmarksTriage only executes the benchmarks on rollup members, that were changed significantly on the rollup.
#163764 e3376ff Tweak suggestion of "unreachable due to never pattern"This perf run didn't have relevant results for the `instruction count` metric.Instruction countThis perf run didn't have relevant results for this metric. Max RSS (memory usage)This perf run didn't have relevant results for this metric. CyclesThis perf run didn't have relevant results for this metric. Binary sizeThis perf run didn't have relevant results for this metric. #164771 62ec256 std: fix platform checks for hermitThis perf run didn't have relevant results for the `instruction count` metric.Instruction countThis perf run didn't have relevant results for this metric. Max RSS (memory usage)This perf run didn't have relevant results for this metric. CyclesThis perf run didn't have relevant results for this metric. Binary sizeThis perf run didn't have relevant results for this metric. #165070 e7363cb create mir for direct-rhs constsInstruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
Max RSS (memory usage)This perf run didn't have relevant results for this metric. CyclesResults (secondary -3.6%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis perf run didn't have relevant results for this metric. #165121 696fc03 test DSL evaluate sequentiallyThis perf run didn't have relevant results for the `instruction count` metric.Instruction countThis perf run didn't have relevant results for this metric. Max RSS (memory usage)This perf run didn't have relevant results for this metric. CyclesThis perf run didn't have relevant results for this metric. Binary sizeThis perf run didn't have relevant results for this metric. #165288 452d7da Add assembly test for masked index without redundant shiftsThis perf run didn't have relevant results for the `instruction count` metric.Instruction countThis perf run didn't have relevant results for this metric. Max RSS (memory usage)This perf run didn't have relevant results for this metric. CyclesThis perf run didn't have relevant results for this metric. Binary sizeThis perf run didn't have relevant results for this metric. #165307 019c532 test(lexer): Add test that ZWJ/ZWNJ are accepted in identifiersThis perf run didn't have relevant results for the `instruction count` metric.Instruction countThis perf run didn't have relevant results for this metric. Max RSS (memory usage)This perf run didn't have relevant results for this metric. CyclesThis perf run didn't have relevant results for this metric. Binary sizeThis perf run didn't have relevant results for this metric. |
Successful merges:
r? @ghost
Create a similar rollup