-
-
Notifications
You must be signed in to change notification settings - Fork 14.4k
Rollup of 4 pull requests #151594
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rollup of 4 pull requests #151594
Conversation
This updates the rust-version file to 63f4513.
Pull recent changes from https://github.com/rust-lang/rust via Josh. Upstream ref: rust-lang/rust@63f4513 Filtered ref: rust-lang/miri@98aa894 Upstream diff: rust-lang/rust@b6fdaf2...63f4513 This merge was created using https://github.com/rust-lang/josh-sync.
Automatic Rustup
This updates the rust-version file to d276646.
Pull recent changes from https://github.com/rust-lang/rust via Josh. Upstream ref: rust-lang/rust@d276646 Filtered ref: rust-lang/miri@e200be9 Upstream diff: rust-lang/rust@63f4513...d276646 This merge was created using https://github.com/rust-lang/josh-sync.
Automatic Rustup
Compile-time variant of shim_sig_arg
This updates the rust-version file to d10ac47.
Pull recent changes from https://github.com/rust-lang/rust via Josh. Upstream ref: rust-lang/rust@d10ac47 Filtered ref: rust-lang/miri@79d95eb Upstream diff: rust-lang/rust@d276646...d10ac47 This merge was created using https://github.com/rust-lang/josh-sync.
Automatic Rustup
…losedir Add test for libc opendir and closedir
update .gitignore to ignore zed files
Unify readdir across Unix targets and add test
readdir: also emit the special directory entries . and ..
windows shims: throw unsupported error for DuplicateHandle on pseudo handle
…e,ytmimi `const` blocks as a `mod` item Tracking issue: rust-lang#149226 This adds support for writing `const { ... }` as an item in a module. In the current implementation, this is a unique AST item that gets lowered to `const _: () = const { ... };` in HIR. rustfmt support included. TODO: - `pub const { ... }` does not make sense (see rust-lang#147136). Reject it. Should this be rejected by the parser or smth? - Improve diagnostics (preferably they should not mention the fake `_` ident).
THIR patterns: Explicitly distinguish `&pin` from plain `&`/`&mut` Currently, `thir::PatKind::Deref` is used for ordinary `&`/`&mut` patterns, and also for `&pin const` and `&pin mut` patterns under `feature(pin_ergonomics)`. The only way to distinguish between them is by inspecting the `Ty` attached to the pattern node. That's non-obvious, making it easy to miss, and is also a bit confusing to read when it does occur. This PR therefore adds an explicit `pin: hir::Pinnedness` field to `thir::PatKind::Deref`, to explicitly distinguish pin-deref nodes from ordinary builtin-deref nodes. (I'm not deeply familiar with the future of pin-patterns, so I'm not sure whether that information is best carried as a field or as a separate `PatKind`, but I think this approach is at least an improvement over the status quo.) r? Nadrieril (or compiler)
miri subtree update Subtree update of `miri` to rust-lang/miri@d4afb41. Created using https://github.com/rust-lang/josh-sync. r? @ghost
…nn,jonathanbrouwer Do not emit errors on non-metaitem diagnostic attr input This unblocks porting over diagnostic attributes by properly addressing rust-lang#151056 (comment) r? @JonathanBrouwer
|
@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: a18e6d9d14 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 a18e6d9 (parent) -> 021fc25 (this PR) Test differencesShow 112 test diffsStage 1
Stage 2
Additionally, 78 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 021fc25b7a48f6051bee1e1f06c7a277e4de1cc9 --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 (021fc25): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)Results (primary -2.6%, secondary 6.5%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 0.7%, secondary 3.3%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 469.207s -> 472.296s (0.66%) |
Successful merges:
constblocks as amoditem #149174 (constblocks as amoditem)&pinfrom plain&/&mut#151282 (THIR patterns: Explicitly distinguish&pinfrom plain&/&mut)r? @ghost
Create a similar rollup