Repository navigation
Rollup of 12 pull requests - #164757
Rollup of 12 pull requests#164757
Conversation
Let's link the relevant UI tests to the Reference's rules for unions.
and update plaform note get illumos sysroot from mirror
This module can contain various helpers for `object`, not strictly those that work with symbols.
Memory64 is merged into the spec since 2025-07-23 (WASM 3.0).
This will make it possible to remove the compiler's Hash impl for FileName.
A subtle consequence of the stage0 redesign is that the compiler must take care to avoid relying on any stdlib-provided Hash impl when computing stable hashes, or at least any such impl that is actually liable to change. This was noticed when trying to strengthen the provided Hash impl for Path, which triggered baffling CI failures because the stable hashes produced by the stage1 compiler (built against stage0 stdlib) were inconsistent with the ones that would have been produced by the stage2 compiler (built against in-tree stdlib). This PR aims to unblock changes to `impl Hash for Path`, by making the compiler not rely on it for stable hashing. It does not attempt to solve the broader problem of the compiler relying on stdlib hash details.
Avoid stdlib's `impl Hash for Path` for stable hashing A subtle consequence of the stage0 redesign is that the compiler must take care to avoid relying on any stdlib-provided Hash impl when computing stable hashes, or at least any such impl that is actually liable to change. This was noticed when trying to strengthen the provided Hash impl for Path, which triggered baffling CI failures because the stable hashes produced by the stage1 compiler (built against stage0 stdlib) were inconsistent with the ones that would have been produced by the stage2 compiler (built against in-tree stdlib). This PR aims to unblock changes to `impl Hash for Path`, by making the compiler not rely on it for stable hashing. It does not attempt to solve the broader problem of the compiler relying on stdlib hash details.
update to illumos sysroot 20210501 Update the sysroot for illumos to this one: https://github.com/illumos/sysroot/releases/#release-20210501-e0b4275f34-v0 Closes rust-lang#150756
rmake: Add helpers for iterating symbols and objects We have a handful tests that do a common pattern of opening a file, looping through its symbols, and performing some validation on each one, which can be a bit inconvenient to do manually. The pattern at https://github.com/rust-lang/compiler-builtins/blob/f521807de24520770991263387ba8c06e462edf8/crates/symcheck/src/main.rs works rather well to make this easier, so add it at `rmake` and update one test that does symbol iteration.
…isDenton refactor Windows remove_dir_all It took me a bit to understand this code. I hope I understood it correctly now. r? @ChrisDenton
…hnTitor std: use `SOCK_NONBLOCK` in `connect_timeout` I doubt the performance improvement will be measurable, but using [`SOCK_NONBLOCK`](https://manned.org/man/socket) upon socket creation instead of manually setting the non-blocking mode seemed like a good idea to me. Implementing this required a consolidation of the `connect_timeout` logic, which seems like a good thing in its own right.
…Borrowck-rename, r=jdonszelmann Rename `PostTypeckUntilBorrowck` -> `PostTypeckThroughBorrowck` This was a rename suggested in the following [Zulip thread](https://rust-lang.zulipchat.com/#narrow/channel/618216-t-types.2Fcall-for-participation/topic/rename.20.60TypingMode.3A.3APostTypeckThroughBorrowck.60/with/624384207) r? jdonszelmann
…x, r=petrochenkov Declare `SyntaxContext` through `rustc_index::newtype_index!` `SyntaxContext` is basically an index type but it is not declared through `newtype_index`. r? @petrochenkov
Avoid cloning CFG-related analysis data. For perf. We can avoid cloning CFG-related data while mutating the body.
…ations_unions, r=PLeVasseur,traviscross Add reference annotations for unions Let's link the relevant UI tests to the Reference's rules for unions.
Allow `llm-assisted` label to be set without write access I.e. allow contributors to set https://github.com/rust-lang/rust/labels/llm-assisted via triagebot as well.
Remove docs mentioning instability of wasm64 Memory64 is merged into the spec, from my reading since 2025-07-23 (and included as part of WebAssembly 3.0). (It still isn't available in Safari and some other engines, but we shouldn't track that in our docs, see https://webassembly.org/features/ for that instead). r? compiler CC target maintainer @alexcrichton
Update LLVM PassBuilder constructor calls to pass a context LLVM's llvm/llvm-project#233150 made the PassBuilder constructor take a LLVMContext argument.
|
💔 Test for eb022b1 failed: CI. Failed job:
|
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
Seems spurious…again. @bors retry |
This comment has been minimized.
This comment has been minimized.
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 7189d39 (parent) -> 0cefa27 (this PR) Test differencesShow 24 test diffsStage 0
Stage 1
Stage 2
Additionally, 19 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 0cefa27fe3deee26fc98ee44249406358d9f0d26 --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 (0cefa27): comparison URL. Overall result: ❌✅ regressions and improvements - please read:Our 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 (primary -0.4%, secondary -0.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 0.1%, secondary 0.5%)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: 486.294s -> 487.555s (0.26%) |
|
📌 Perf builds for each rolled up PR:
parent commit: 7189d39d9c In the case of a perf regression, run the following command with the SHAs of each PR you suspect might be the cause: |
|
@rust-timer triage all |
Running triage with 6 benchmarksTriage only executes the benchmarks on rollup members, that were changed significantly on the rollup.
#164458 58d0037 Avoid stdlib's
|
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
- | - | 0 |
| Regressions ❌ (secondary) |
0.4% | [0.3%, 0.5%] | 5 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
- | - | 0 |
| All ❌✅ (primary) | - | - | 0 |
Max RSS (memory usage)
This perf run didn't have relevant results for this metric.
Cycles
This perf run didn't have relevant results for this metric.
Binary size
Results (primary -0.1%)
A less reliable metric. May be of interest, but not used to determine the overall result above.
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
- | - | 0 |
| Regressions ❌ (secondary) |
- | - | 0 |
| Improvements ✅ (primary) |
-0.1% | [-0.1%, -0.0%] | 4 |
| Improvements ✅ (secondary) |
- | - | 0 |
| All ❌✅ (primary) | -0.1% | [-0.1%, -0.0%] | 4 |
#161790 8866e85 update to illumos sysroot 20210501
This perf run didn't have relevant results for the `instruction count` metric.
Instruction count
This perf run didn't have relevant results for this metric.
Max RSS (memory usage)
Results (secondary -1.4%)
A less reliable metric. May be of interest, but not used to determine the overall result above.
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
- | - | 0 |
| Regressions ❌ (secondary) |
- | - | 0 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
-1.4% | [-1.4%, -1.4%] | 1 |
| All ❌✅ (primary) | - | - | 0 |
Cycles
This perf run didn't have relevant results for this metric.
Binary size
This perf run didn't have relevant results for this metric.
#163087 e830d43 rmake: Add helpers for iterating symbols and objects
This perf run didn't have relevant results for the `instruction count` metric.
Instruction count
This perf run didn't have relevant results for this metric.
Max RSS (memory usage)
Results (secondary 4.9%)
A less reliable metric. May be of interest, but not used to determine the overall result above.
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
- | - | 0 |
| Regressions ❌ (secondary) |
4.9% | [4.9%, 4.9%] | 1 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
- | - | 0 |
| All ❌✅ (primary) | - | - | 0 |
Cycles
This perf run didn't have relevant results for this metric.
Binary size
This perf run didn't have relevant results for this metric.
#163586 cffb528 refactor Windows remove_dir_all
This perf run didn't have relevant results for the `instruction count` metric.
Instruction count
This perf run didn't have relevant results for this metric.
Max RSS (memory usage)
Results (secondary -1.6%)
A less reliable metric. May be of interest, but not used to determine the overall result above.
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
- | - | 0 |
| Regressions ❌ (secondary) |
- | - | 0 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
-1.6% | [-1.6%, -1.6%] | 1 |
| All ❌✅ (primary) | - | - | 0 |
Cycles
This perf run didn't have relevant results for this metric.
Binary size
This perf run didn't have relevant results for this metric.
#163983 f0692c9 std: use SOCK_NONBLOCK in connect_timeout
This perf run didn't have relevant results for the `instruction count` metric.
Instruction count
This perf run didn't have relevant results for this metric.
Max RSS (memory usage)
Results (secondary 1.5%)
A less reliable metric. May be of interest, but not used to determine the overall result above.
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
- | - | 0 |
| Regressions ❌ (secondary) |
4.8% | [4.8%, 4.8%] | 1 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
-1.9% | [-1.9%, -1.9%] | 1 |
| All ❌✅ (primary) | - | - | 0 |
Cycles
This perf run didn't have relevant results for this metric.
Binary size
This perf run didn't have relevant results for this metric.
#164307 e69fad2 Rename PostTypeckUntilBorrowck -> PostTypeckThroughBorrowck
This perf run didn't have relevant results for the `instruction count` metric.
Instruction count
This perf run didn't have relevant results for this metric.
Max RSS (memory usage)
This perf run didn't have relevant results for this metric.
Cycles
This perf run didn't have relevant results for this metric.
Binary size
This perf run didn't have relevant results for this metric.
#164425 58a325d Declare SyntaxContext through rustc_index::newtype_index!
Instruction count
Our most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
- | - | 0 |
| Regressions ❌ (secondary) |
0.3% | [0.3%, 0.3%] | 2 |
| Improvements ✅ (primary) |
-0.2% | [-0.2%, -0.2%] | 3 |
| Improvements ✅ (secondary) |
-0.4% | [-0.7%, -0.3%] | 15 |
| All ❌✅ (primary) | -0.2% | [-0.2%, -0.2%] | 3 |
Max RSS (memory usage)
Results (secondary -5.8%)
A less reliable metric. May be of interest, but not used to determine the overall result above.
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
- | - | 0 |
| Regressions ❌ (secondary) |
- | - | 0 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
-5.8% | [-5.8%, -5.8%] | 1 |
| All ❌✅ (primary) | - | - | 0 |
Cycles
Results (secondary -15.5%)
A less reliable metric. May be of interest, but not used to determine the overall result above.
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
- | - | 0 |
| Regressions ❌ (secondary) |
- | - | 0 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
-15.5% | [-15.5%, -15.5%] | 1 |
| All ❌✅ (primary) | - | - | 0 |
Binary size
This perf run didn't have relevant results for this metric.
#156841 ced830a Avoid cloning CFG-related analysis data.
This perf run didn't have relevant results for the `instruction count` metric.
Instruction count
This perf run didn't have relevant results for this metric.
Max RSS (memory usage)
Results (secondary -1.4%)
A less reliable metric. May be of interest, but not used to determine the overall result above.
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
- | - | 0 |
| Regressions ❌ (secondary) |
- | - | 0 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
-1.4% | [-1.4%, -1.4%] | 1 |
| All ❌✅ (primary) | - | - | 0 |
Cycles
This perf run didn't have relevant results for this metric.
Binary size
This perf run didn't have relevant results for this metric.
#164654 a605b15 Add reference annotations for unions
This perf run didn't have relevant results for the `instruction count` metric.
Instruction count
This perf run didn't have relevant results for this metric.
Max RSS (memory usage)
Results (secondary -3.6%)
A less reliable metric. May be of interest, but not used to determine the overall result above.
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
- | - | 0 |
| Regressions ❌ (secondary) |
- | - | 0 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
-3.6% | [-8.4%, -1.0%] | 6 |
| All ❌✅ (primary) | - | - | 0 |
Cycles
Results (secondary 4.1%)
A less reliable metric. May be of interest, but not used to determine the overall result above.
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
- | - | 0 |
| Regressions ❌ (secondary) |
4.1% | [2.9%, 5.4%] | 2 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
- | - | 0 |
| All ❌✅ (primary) | - | - | 0 |
Binary size
This perf run didn't have relevant results for this metric.
#164717 c913f2c Allow llm-assisted label to be set without write access
This perf run didn't have relevant results for the `instruction count` metric.
Instruction count
This perf run didn't have relevant results for this metric.
Max RSS (memory usage)
This perf run didn't have relevant results for this metric.
Cycles
This perf run didn't have relevant results for this metric.
Binary size
This perf run didn't have relevant results for this metric.
#164735 00141e3 Remove docs mentioning instability of wasm64
This perf run didn't have relevant results for the `instruction count` metric.
Instruction count
This perf run didn't have relevant results for this metric.
Max RSS (memory usage)
Results (secondary -2.0%)
A less reliable metric. May be of interest, but not used to determine the overall result above.
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
- | - | 0 |
| Regressions ❌ (secondary) |
- | - | 0 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
-2.0% | [-2.4%, -1.7%] | 2 |
| All ❌✅ (primary) | - | - | 0 |
Cycles
This perf run didn't have relevant results for this metric.
Binary size
This perf run didn't have relevant results for this metric.
#164737 e8146b0 Update LLVM PassBuilder constructor calls to pass a context
This perf run didn't have relevant results for the `instruction count` metric.
Instruction count
This perf run didn't have relevant results for this metric.
Max RSS (memory usage)
Results (secondary -2.1%)
A less reliable metric. May be of interest, but not used to determine the overall result above.
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
- | - | 0 |
| Regressions ❌ (secondary) |
- | - | 0 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
-2.1% | [-2.1%, -2.1%] | 1 |
| All ❌✅ (primary) | - | - | 0 |
Cycles
This perf run didn't have relevant results for this metric.
Binary size
This perf run didn't have relevant results for this metric.
View all comments
Successful merges:
impl Hash for Pathfor stable hashing #164458 (Avoid stdlib'simpl Hash for Pathfor stable hashing)SOCK_NONBLOCKinconnect_timeout#163983 (std: useSOCK_NONBLOCKinconnect_timeout)PostTypeckUntilBorrowck->PostTypeckThroughBorrowck#164307 (RenamePostTypeckUntilBorrowck->PostTypeckThroughBorrowck)SyntaxContextthroughrustc_index::newtype_index!#164425 (DeclareSyntaxContextthroughrustc_index::newtype_index!)llm-assistedlabel to be set without write access #164717 (Allowllm-assistedlabel to be set without write access)Failed merges:
record_*macros from the metadata encoder #164636 (Removerecord_*macros from the metadata encoder)r? @ghost
Create a similar rollup