-
-
Notifications
You must be signed in to change notification settings - Fork 14.4k
Rollup of 6 pull requests #151459
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 6 pull requests #151459
Conversation
…nore-*` We want directives nice and tidy.
…, `no-SingleUseConsts-mir-pass` To prevent regressions our test must cover the code both inside and outside of the `SingleUseConsts` MIR pass. Use revisions for that.
This completes the transition to annotate-snippets
tests/debuginfo/basic-stepping.rs: Add revisions `default-mir-passes`, `no-SingleUseConsts-mir-pass` To prevent regressions our test must cover the code both inside and outside of the `SingleUseConsts` MIR pass. Use revisions for that. We know this use case is sensitive to regressions because it already happened at least once. See rust-lang#33013 (comment). CC rust-lang#130896
…=nnethercote Remove old error emitter This completes the transition to annotate-snippets and cuts 3600 lines of code.
…, r=RalfJung use `deny(missing_docs)` for non-mir intrinsics
Move UI tests Moved several regression tests from `tests/ui/issues` to specific folders
Derive `Default` for `QueryArenas` There's no need to manually implement Default for this struct, because the fields are all `TypeArena<_>` or `()`, which both implement Default already. This lets us avoid one occurrence of the `query_if_arena!` macro.
make `simd_insert_dyn` and `simd_extract_dyn` const For use in `stdarch`. We currently use an equivalent of the fallback body here, but on some targets the intrinsic generate better code. r? RalfJung
|
@bors r+ rollup=never p=5 |
This comment has been minimized.
This comment has been minimized.
|
📌 Perf builds for each rolled up PR:
previous master: 625b63f9e1 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 625b63f (parent) -> daa90b9 (this PR) Test differencesShow 75 test diffsStage 1
Stage 2
Additionally, 30 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 daa90b96dd039ce0c5708408972e2af45990c1db --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 |
|
Finished benchmarking commit (daa90b9): comparison URL. Overall result: ❌ regressions - 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%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 2.1%)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: 474.946s -> 470.203s (-1.00%) |
Successful merges:
default-mir-passes,no-SingleUseConsts-mir-pass#147426 ( tests/debuginfo/basic-stepping.rs: Add revisionsdefault-mir-passes,no-SingleUseConsts-mir-pass)deny(missing_docs)for non-mir intrinsics #151432 (usedeny(missing_docs)for non-mir intrinsics)DefaultforQueryArenas#151445 (DeriveDefaultforQueryArenas)simd_insert_dynandsimd_extract_dynconst #151453 (makesimd_insert_dynandsimd_extract_dynconst)r? @ghost
Create a similar rollup