Skip to content
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 8 pull requests #137608

Merged
merged 19 commits into from
Feb 25, 2025
Merged

Rollup of 8 pull requests #137608

merged 19 commits into from
Feb 25, 2025

Conversation

fmease
Copy link
Member

@fmease fmease commented Feb 25, 2025

Successful merges:

r? @ghost
@rustbot modify labels: rollup

Create a similar rollup

RalfJung and others added 19 commits February 21, 2025 12:21
This refers to a `PredicatePolarity::Reserved` which no longer exists.
The force inlining attribute isn't is never used with `#![..]` attribute
syntax, only `#[..]` syntax.
…owLii

adjust_abi: make fallback logic for ABIs a bit easier to read

I feel like the match guards here make this unnecessarily harder to follow.
Improve behavior of `IF_LET_RESCOPE` around temporaries and place expressions

Heavily reworks the `IF_LET_RESCOPE` to be more sensitive around 1. temporaries that get consumed/terminated and therefore should not trigger the lint, and 2. borrows of place expressions, which are not temporary values.

Fixes rust-lang#137411
…tebank

Fix invalid suggestion from type error for derive macro

Fixes rust-lang#136343

r? `@estebank`

I didn't use `from_expansion` to avoid it because of testcase
`tests/ui/typeck/issue-110017-format-into-help-deletes-macro.rs`:

https://github.com/chenyukang/rust/blob/11959a8b6e75d2c55500a703070a248342d29549/tests/ui/typeck/issue-110017-format-into-help-deletes-macro.rs#L34-L37

This type error could come up with a proper fix.
… r=notriddle

 Add rustdoc-gui regression test for rust-lang#137082

Fixes rust-lang#137082.

Added new commands in `browser-ui-test` allowing us to add a regression test for rust-lang#137082 and also another to copy code examples content.

r? `@notriddle`
Don't doc-comment BTreeMap<K, SetValZST, A>

This otherwise shows up in documentation as an empty impl block (worse, at the *top* of the docs above the public impls).
…=RalfJung

remove `simd_fpow` and `simd_fpowi`

Discussed in rust-lang#137555

These functions are not exposed from `std::intrinsics::simd`, and not used anywhere outside of the compiler. They also don't lower to particularly good code at least on the major ISAs (I checked x86_64, aarch64, s390x, powerpc), where the vector is just spilled to the stack and scalar functions are used for the actual logic.

r? `@RalfJung`
…nt, r=fmease

type_ir: remove redundant part of comment

This comment refers to a `PredicatePolarity::Reserved` which no longer exists.
…, r=fmease

feature: fix typo in attribute description

The force inlining attribute isn't is never used with `#![..]` attribute syntax, only `#[..]` syntax.
@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. T-rustdoc-frontend Relevant to the rustdoc-frontend team, which will review and decide on the web UI/UX output. rollup A PR which is a rollup labels Feb 25, 2025
@fmease
Copy link
Member Author

fmease commented Feb 25, 2025

@bors r+ rollup=never p=5

@bors
Copy link
Contributor

bors commented Feb 25, 2025

📌 Commit 9fda3e6 has been approved by fmease

It is now in the queue for this repository.

@bors bors removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 25, 2025
@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Feb 25, 2025
@bors
Copy link
Contributor

bors commented Feb 25, 2025

⌛ Testing commit 9fda3e6 with merge 85abb27...

@bors
Copy link
Contributor

bors commented Feb 25, 2025

☀️ Test successful - checks-actions
Approved by: fmease
Pushing 85abb27 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Feb 25, 2025
@bors bors merged commit 85abb27 into rust-lang:master Feb 25, 2025
7 checks passed
@rustbot rustbot added this to the 1.87.0 milestone Feb 25, 2025
@fmease fmease deleted the rollup-h4siso6 branch February 25, 2025 22:59
@rust-timer
Copy link
Collaborator

📌 Perf builds for each rolled up PR:

PR# Message Perf Build Sha
#137370 adjust_abi: make fallback logic for ABIs a bit easier to re… 03f0c93fc6c0a75413e7f4ebe53a151ea6f6b49e (link)
#137444 Improve behavior of IF_LET_RESCOPE around temporaries and… 845d32d747af7801a04a55f879651347671d71b7 (link)
#137464 Fix invalid suggestion from type error for derive macro b82b1d5c73c45074da400fa1d6730cff892dbc20 (link)
#137539 Add rustdoc-gui regression test for #137082 065ec0e3f06b0a7707dc7ee2aa8b6d6721123371 (link)
#137576 Don't doc-comment BTreeMap<K, SetValZST, A> 2e70014f562c5c062720424f768257745f176bf8 (link)
#137595 remove simd_fpow and simd_fpowi 43cd4c6c7f609f8b949a03102ef2055e33715c76 (link)
#137600 type_ir: remove redundant part of comment d4468d3aba296bb901bbdae30face0611f8be58a (link)
#137602 feature: fix typo in attribute description 08fb175c51ad77d98f1eb455b146ff49c54696c6 (link)

previous master: 4ecd70ddd1

In the case of a perf regression, run the following command for each PR you suspect might be the cause: @rust-timer build $SHA

@rust-timer
Copy link
Collaborator

Finished benchmarking commit (85abb27): comparison URL.

Overall result: no relevant changes - no action needed

@rustbot label: -perf-regression

Instruction count

This benchmark run did not return any relevant results for this metric.

Max RSS (memory usage)

Results (secondary -3.1%)

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

mean range count
Regressions ❌
(primary)
- - 0
Regressions ❌
(secondary)
- - 0
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
-3.1% [-4.3%, -1.9%] 2
All ❌✅ (primary) - - 0

Cycles

Results (secondary 2.3%)

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

mean range count
Regressions ❌
(primary)
- - 0
Regressions ❌
(secondary)
2.3% [2.3%, 2.3%] 1
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
- - 0
All ❌✅ (primary) - - 0

Binary size

This benchmark run did not return any relevant results for this metric.

Bootstrap: 770.644s -> 770.518s (-0.02%)
Artifact size: 361.94 MiB -> 362.00 MiB (0.02%)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc merged-by-bors This PR was explicitly merged by bors. rollup A PR which is a rollup S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. T-rustdoc-frontend Relevant to the rustdoc-frontend team, which will review and decide on the web UI/UX output.
Projects
None yet
Development

Successfully merging this pull request may close these issues.