-
-
Notifications
You must be signed in to change notification settings - Fork 14.4k
Rollup of 3 pull requests #150582
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 3 pull requests #150582
Conversation
It allows to get rid of `AmbiguityErrorMisc` and `Flags`.
Added a `# Panics` section to `Iterator::last` to match `Iterator::count`, after the change made in Rust 1.92.0 where now `std::iter::Repeat` panics in these methods instead of doing an infinite `loop`. Cites rust-lang#149707
…erimental-arch, r=nnethercote Make `asm_experimental_arch` work in `allow_internal_unstable` macros This change makes it possible to use unstable `asm!`, usually requiring `feature(asm_experimental_arch)`, in proc-macros with the `allow_internal_unstable` attribute. The test was added on a target where `asm!` is unstable: Wasm. However, this affects *any* target with an unstable `asm!` implementation.
resolve: Preserve binding scopes in ambiguity errors It allows to get rid of `AmbiguityErrorMisc`, `Flags` and `extern_prelude_flag_binding`. Also keep all encountered bindings in `resolve_ident_in_scope_set`, it allows to get rid of `extern_prelude_item_binding`. This also unblocks rust-lang#149681 because the newly preserved data will help to fix the regressions in rust-lang#149681 (comment) in a reasonable way.
Document `panic!` in `Iterator::last` for rust-lang#149707 This PR adds a section `# Panics` to the method `Iterator::last` equal to the section `# Panics` of the method `Iterator::count`, because now, after the changes made in Rust 1.92.0, both methods in `std::iter::Repeat` panics instead of doing an infinite loop. `Iterator::count` already has documentation for this behavior ("This function might panic if the iterator has more than `usize::MAX` elements."), but `Iterator::last` does not. This PR adds documentation for this panic in the same way as `Iterator::count` does. Issue: rust-lang#149707
|
@bors r+ rollup=never p=3 |
|
☀️ Test successful - checks-actions |
|
📌 Perf builds for each rolled up PR:
previous master: a47f4dfd7f 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 a47f4df (parent) -> 1b43252 (this PR) Test differencesShow 588 test diffsStage 1
Stage 2
Additionally, 586 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 1b4325211c3228dd10eea12e9b3f749f87363d95 --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 (1b43252): 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 (primary 0.3%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 0.5%, secondary -2.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (primary 0.0%, secondary 0.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 473.624s -> 472.736s (-0.19%) |
|
Tiny changes, overall it's a wash. @rustbot label: +perf-regression-triaged |
Successful merges:
asm_experimental_archwork inallow_internal_unstablemacros #150073 (Makeasm_experimental_archwork inallow_internal_unstablemacros)panic!inIterator::lastfor #149707 #150580 (Documentpanic!inIterator::lastfor Documentpanic!for Iterator::last (Repeat) #149707)r? @ghost
@rustbot modify labels: rollup
Create a similar rollup