-
Notifications
You must be signed in to change notification settings - Fork 13.8k
Rollup of 5 pull requests #146409
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rollup of 5 pull requests #146409
Conversation
v6.17-rc5 contains the equivalent of the two commits we had here, thus move the Rust for Linux job forward to that so that we don't need the temporary commits anymore. Signed-off-by: Miguel Ojeda <[email protected]>
Keeping this up-to-date as the project itself, and its dependencies, are updated.
We need a different attribute than `rustc_align` because unstable attributes are tied to their feature (we can't have two unstable features use the same unstable attribute). Otherwise this uses all of the same infrastructure as `#[rustc_align]`.
…hpratt inclusive `Range`s: change `end` to `last` Tracking issue: rust-lang#125687 ACP: rust-lang/libs-team#511
…lmann,ralfjung,traviscross Implement `#[rustc_align_static(N)]` on `static`s Tracking issue: rust-lang#146177 ```rust #![feature(static_align)] #[rustc_align_static(64)] static SO_ALIGNED: u64 = 0; ``` We need a different attribute than `rustc_align` because unstable attributes are tied to their feature (we can't have two unstable features use the same unstable attribute). Otherwise this uses all of the same infrastructure as `#[rustc_align]`. r? `@traviscross`
CI: rfl: move job forward to Linux v6.17-rc5 to remove temporary commits v6.17-rc5 contains the equivalent of the two commits we had here, thus move the Rust for Linux job forward to that so that we don't need the temporary commits anymore. r? ```@lqd``` ```@Kobzol``` try-job: x86_64-rust-for-linux ```@rustbot``` label A-rust-for-linux ```@bors``` try
…-ld, r=lqd Update wasm-component-ld to 0.5.17 Keeping this up-to-date as the project itself, and its dependencies, are updated.
Trim paths less in MIR dumping With this PR, the paths MIR dump filters and that are printed at the start of a dump file are no longer trimmed. They don't include the crate that is being compiled, however.
@bors r+ rollup=never p=5 |
Rollup of 5 pull requests Successful merges: - #144765 (inclusive `Range`s: change `end` to `last`) - #146178 (Implement `#[rustc_align_static(N)]` on `static`s) - #146368 (CI: rfl: move job forward to Linux v6.17-rc5 to remove temporary commits) - #146378 (Update wasm-component-ld to 0.5.17) - #146391 (Trim paths less in MIR dumping) r? `@ghost` `@rustbot` modify labels: rollup
The job Click to see the possible cause of the failure (guessed by this bot)
|
💔 Test failed - checks-actions |
@bors retry |
☀️ Test successful - checks-actions |
📌 Perf builds for each rolled up PR:
previous master: 7ad23f43a2 In the case of a perf regression, run the following command for each PR you suspect might be the cause: |
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 7ad23f4 (parent) -> 565a9ca (this PR) Test differencesShow 132 test diffsStage 0
Stage 1
Stage 2
Additionally, 114 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 565a9ca63e9df4b223fed0da01f15e578acfb538 --output-dir test-dashboard And then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
Finished benchmarking commit (565a9ca): 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 (secondary 0.5%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary 5.3%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 469.26s -> 468.891s (-0.08%) |
perf triage: Some small regressions. I suspect the primary regressions in Either way, together with the secondary benchmarks, all of those are tiny changes (0.002s) on tiny benchmarks, so I don't think this worth more investigation. @rustbot label: +perf-regression-triaged |
Successful merges:
Range
s: changeend
tolast
#144765 (inclusiveRange
s: changeend
tolast
)#[rustc_align_static(N)]
onstatic
s #146178 (Implement#[rustc_align_static(N)]
onstatic
s)r? @ghost
@rustbot modify labels: rollup
Create a similar rollup