Rollup of 8 pull requests#153377
Conversation
Add a convenience method Path::absolute() that delegates to the existing free function std::path::absolute(), mirroring the pattern of Path::canonicalize() delegating to fs::canonicalize(). Tracking issue: rust-lang#153328
rephrasing and grammar
…r=lolbinarycat Add regression test for `doc(fake_variadic)` on reexports Follow-up of rust-lang#153136. Out of the four doc attributes remaining to be tested (`fake_variadic`, `keyword`, `attribute` and `notable_trait`), only `fake_variadic` could have been impacted by reexports: * `attribute` and `keyword` are not supposed to be reexported (they're supposed to be used on private modules) * `notable_trait` is applied to a trait, so whether it's reexported or not, it still works. r? @lolbinarycat
x86: reserve `bl` and `bh` registers to match `rbx` `bl` and `bh` need to be explicitly marked as reserved, as they are sub-registers of `rbx`, which is reserved by LLVM. Discovered this while trying to run Graviola through Cranelift, which was becoming corrupted due to the register allocator assigning `bl`: rust-lang/rustc_codegen_cranelift#1629 cc: @bjorn3
…s, r=tgross35 Boundary tests for various Duration-float operations As requested in rust-lang#150933 (comment) r? tgross35
…t-else-wording, r=Kivooeo Improve irrefutable let-else lint wording Update the `irrefutable_let_patterns` wording for let-else to better reflect that the `else` clause is unreachable when the LHS pattern always matches. Closes rust-lang#152938
diag: Suppress `.clone()` suggestion inside derive macro expansions Derives on a packed struct with non-Copy fields generate moves whose spans point back at the original field type definitions. `suggest_cloning_inner()` then proposes inserting `.clone()` at those locations, producing nonsensical output like `String.clone()` in the struct definition. close rust-lang#153126
…=tgross35 Add `Path::absolute` method as alias for `std::path::absolute` `Path::canonicalize()` is an alias for `fs::canonicalize()`, but there's no equivalent `Path::absolute()` for `path::absolute()`. This adds one. Discussed in rust-lang#92750 (comment), Chris [said](rust-lang#92750 (comment)) a PR would be welcome. Tracking issue: rust-lang#153328
…r=tgross35 update panicking() docs for panic=abort fixes rust-lang#151458 The documentation for `std::thread::panicking()` has not been changed since v1.0, even though panic hooks were added in v1.10. Current documentation is misleading for `panic=abort` `panicking()` can return `true` in 2 different cases: 1. Thread unwinds due to panic 2. Panic hook is executing with `panic=abort` r? @SpriteOvO
…athanBrouwer Migration of `LintDiagnostic` - part 6 Part of rust-lang#153099. r? @JonathanBrouwer
|
Rollup of everything. @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: d9563937fa 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 d956393 (parent) -> d933cf4 (this PR) Test differencesShow 152 test diffsStage 1
Stage 2
Additionally, 124 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 d933cf483edf1605142ac6899ff32536c0ad8b22 --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 (d933cf4): 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 6.2%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary -1.6%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (primary -0.0%, secondary -0.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 480.491s -> 481.183s (0.14%) |
|
Missing artifact for sha |
|
@rust-timer build f01a365 |
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (f01a365): comparison URL. Overall result: ❌ regressions - no action neededInstruction 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.3%)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: 480.491s -> 480.412s (-0.02%) |
|
@rust-timer build 7437730 |
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (7437730): comparison URL. Overall result: ❌✅ regressions and improvements - no action neededInstruction 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.2%)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 sizeResults (primary -0.0%, secondary -0.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 480.491s -> 479.963s (-0.11%) |
Successful merges:
doc(fake_variadic)on reexports #153280 (Add regression test fordoc(fake_variadic)on reexports)blandbhregisters to matchrbx#153302 (x86: reserveblandbhregisters to matchrbx).clone()suggestion inside derive macro expansions #153258 (diag: Suppress.clone()suggestion inside derive macro expansions)Path::absolutemethod as alias forstd::path::absolute#153272 (AddPath::absolutemethod as alias forstd::path::absolute)LintDiagnostic- part 6 #153352 (Migration ofLintDiagnostic- part 6)r? @ghost
Create a similar rollup