Rollup of 4 pull requests - #162362
Conversation
The unsigned counterparts got the attribute in #724fe8f, but the signed side was missed, so panics like `0i32.wrapping_div(0)` point at the library source instead of the caller. Also covers overflowing_div_euclid, which the issue missed but wrapping_div_euclid delegates to, so it would still report the wrong location without it.
The previous documentation was, in my opinion, overly focused on the implementation of these items, rather than how and when to use them. This new documentation: * Begins the `format_into()` documentation with what it does when called. * Specifies that `format_into()` is a specialized alternative to the more commonly used `Display`. * Specifies that `NumBuffer` cannot be read or otherwise used by itself.
…ke_canonical_response
…uVanilla remove more trivial regions in evaluate_added_goals_and_make_canonical_response cc rust-lang#161575. I don't think it *fixes* the issue per se, since the most principled fix would be to just deny unconstrained lifetime args like we do for types and consts already. At the very least, though, it Makes Things Go Faster. In the example from rust-lang#161575 (comment), each `'unconstrained` appears only once in the entire response: in the rhs of the `'a: 'unconstrained#N` bound. Since they are mentioned nowhere else and are created only when proving our own nested goals, these outlives constraints are all satisfiable by setting `'unconstrained := 'empty`, which tells us nothing about `'a`. Therefore, (I think) that makes it safe to treat all of these constraints as trivial and drop them entirely, drop all of these requirements entirely, similar to what we already do with reflexive or duplicate region constraints. In other words, if a `ReVar` appears only once in the entire response, and that place is the rhs of an outlives constraint, then it is safe to drop that constraint. I'm a little worried about the perf impact of the visitor on "normal" code, but fwiw even a 100-deep nested version of the reproducer compiles in about 0.05s on my machine. r? lcnr
…div, r=Mark-Simulacrum Add #[track_caller] to signed integer div/rem methods Fixes rust-lang#139672 The unsigned div/rem methods got #[track_caller] in 724fe8f but the signed side was missed. This adds it to the 8 signed methods listed in the issue, including overflowing_div_euclid which the issue missed but wrapping_div_euclid delegates to. No test added, matching the unsigned side. cg_gcc doesn't propagate track_caller through inlined div/rem yet so a runtime check would fail on the gcc-core-tests CI job.
…obzol Revert "bootstrap: Enable rustdoc mergeable CCI for std and internal docs rust-lang#161716" cc rust-lang#162334. It seems that rust-lang#161716 still has some issues with combining cross-crate info? Let's revert for now and reland once we figure out why we are regressing the search results 🤔 It's not obvious to me. I built `rustc-docs` dist component locally, and at least locally with rust-lang#161716 reverted I can find `rustc::ast::Attribute` "normally" from the search results. This reverts commit a69a632, reversing changes made to c33d8f3. This also reverts rust-lang#162318, since rust-lang#162318 fixes an regression introduced in rust-lang#162334. r? @Kobzol (or notriddle/bootstrap/rustdoc) cc @notriddle
Revise documentation of `NumBuffer` and `format_into()`. The previous documentation was, in my opinion, overly focused on the implementation of these items, rather than how and when to use them. In particular, this new documentation: * Begins the `format_into()` documentation with what it does when called. * Specifies that `format_into()` is a specialized alternative to the more commonly used `Display`. * Specifies that `NumBuffer` cannot be read or otherwise used by itself. @rustbot label +A-docs
|
⌛ Trying commit 3b6ec47 with merge 2519bbe… To cancel the try build, run the command Workflow: https://github.com/rust-lang/rust/actions/runs/34024390054 |
Rollup of 4 pull requests try-job: dist-various-1 try-job: test-various try-job: x86_64-gnu-aux try-job: x86_64-gnu-llvm-21-3 try-job: x86_64-msvc-1 try-job: aarch64-apple-1 try-job: aarch64-apple-2 try-job: x86_64-mingw-1 try-job: i686-msvc-1 try-job: i686-msvc-2
|
@bors try cancel |
This comment has been minimized.
This comment has been minimized.
|
Try build cancelled. Cancelled workflows: Hint: if you want to run another try build, you do not need to manually cancel the previous one. Just run |
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 46a266b (parent) -> 7cef43f (this PR) Test differencesShow 1811 test diffsStage 0
Stage 1
Stage 2
Additionally, 1806 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 7cef43fbb6862a20766f206ed6fec78124922d2c --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 |
|
📌 Perf builds for each rolled up PR:
parent commit: 46a266b9fb In the case of a perf regression, run the following command for each PR you suspect might be the cause: |
|
Finished benchmarking commit (7cef43f): comparison URL. Overall result: ❌✅ regressions and improvements - 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 -1.4%, secondary -0.8%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary -2.3%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis perf run didn't have relevant results for this metric. Bootstrap: 478.283s -> 478.023s (-0.05%) |
Successful merges:
NumBufferandformat_into(). #162158 (Revise documentation ofNumBufferandformat_into().)r? @ghost
Create a similar rollup