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