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 |
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