Repository navigation
Rollup of 3 pull requests - #164117
Rollup of 3 pull requests#164117
Conversation
libc-pipe test: fix FreeBSD-related comment
On FreeBSD this may cause trouble if the ABI does not match between libc versions. However, std and libc currently use the same ABI version. If this ever changes, we should figure out how FreeBSD supports old programs against a new OS, and do the same thing.
stop trusting the user types for stat and dirent
These three dependencies are only used on x86 targets causing `cargo::unused_dependencies` warnings on other architectures.
narrow nix/ipc-channel/capstone deps to x86 targets
Windows CreateFileW: detect some more unsupported flag combinations
ui tests: assume less about user paths
add openat support
stub out some shims to make File debug-print something
CI: reject all warnings, including those from cargo
This updates the rust-version file to 0f6e5bf.
Pull recent changes from https://github.com/rust-lang/rust via Josh. Previous upstream ref: rust-lang/rust@56343b1 New upstream ref: rust-lang/rust@0f6e5bf Filtered ref: rust-lang/miri@5e5a38d Upstream diff: rust-lang/rust@56343b1...0f6e5bf This merge was created using https://github.com/rust-lang/josh-sync.
Automatic Rustup
test some more openat uses and fix Windows symlink handling
miri subtree update In particular this fixes some unused dependencies. --- Subtree update of `miri` to rust-lang/miri@be95537. Created using https://github.com/rust-lang/josh-sync. r? @ghost
…a-kruppe Replace `Unique` in `Box` with a `(NonNull, PhantomData)` wrapper Follow-up to rust-lang#162804 See [zulip](https://rust-lang.zulipchat.com/#narrow/channel/219381-t-libs/topic/Can.20we.20nuke.20.60Unique.60.3F/with/624141418). This is only the first step in actually refactoring `Box`, and is essentially just a rename. The layout of `Box` stays as-is for now, due to rust-lang#162850. Needs perf run
…, r=jieyouxu Disable default cc optimizations in shell.nix Fixes rust-lang/miri#5393. Apparently, in dev shells, nix configures the c compiler to pass `-O2` by default for hardening. This is a problem because it breaks some miri tests, since miri doesn't support all instructions in its native-lib handling, and thus needs the c compiler to behave somewhat predictably (not enable optimizations it didn't ask for). This PR disables that particular hardening mechansim in the `shell.nix`, so that the c compiler behaves like everyone expects a normal c compiler to behave. Relevant part of the nix docs: https://nixos.org/manual/nixpkgs/unstable/#fortify After this PR, `x test miri -- tests/native-lib/pass/ptr_write_access.rs` runs successfully inside the dev shell. I've verified this both on my nixos install, as well as on a minimal fedora vm. r? bootstrap
|
@bors r+ p=5 force |
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 6e4cba3 (parent) -> 6f5da40 (this PR) Test differencesShow 243 test diffs243 doctest diffs were found. These are ignored, as they are noisy. Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard 6f5da406b56c2db6607202b63daee73aa8c61297 --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: 6e4cba3380 In the case of a perf regression, run the following command with the SHAs of each PR you suspect might be the cause: |
|
Finished benchmarking commit (6f5da40): comparison URL. Overall result: ❌ regressions - 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.3%, secondary 0.2%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesThis perf run didn't have relevant results for this metric. Binary sizeResults (primary 0.1%, secondary 0.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 488.311s -> 488.596s (0.06%) |
|
@rust-timer triage all |
Running triage with 2 benchmarksTriage only executes the benchmarks on rollup members, that were changed significantly on the rollup.
#164092 c99482c miri subtree updateThis perf run didn't have relevant results for the `instruction count` metric.Instruction countThis 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. CyclesThis perf run didn't have relevant results for this metric. Binary sizeResults (primary 0.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
#162849 9404582 Replace
|
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
0.4% | [0.2%, 0.5%] | 3 |
| Regressions ❌ (secondary) |
- | - | 0 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
- | - | 0 |
| All ❌✅ (primary) | 0.4% | [0.2%, 0.5%] | 3 |
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.1% | [0.1%, 0.1%] | 12 |
| Regressions ❌ (secondary) |
- | - | 0 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
- | - | 0 |
| All ❌✅ (primary) | 0.1% | [0.1%, 0.1%] | 12 |
#163900 04793d9 Disable default cc optimizations in shell.nix
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
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.1% | [0.1%, 0.1%] | 12 |
| Regressions ❌ (secondary) |
- | - | 0 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
- | - | 0 |
| All ❌✅ (primary) | 0.1% | [0.1%, 0.1%] | 12 |
Successful merges:
UniqueinBoxwith a(NonNull, PhantomData)wrapper #162849 (ReplaceUniqueinBoxwith a(NonNull, PhantomData)wrapper)r? @ghost
Create a similar rollup