Skip to content

Conversation

matthiaskrgr
Copy link
Member

Successful merges:

r? @ghost
@rustbot modify labels: rollup

Create a similar rollup

tgross35 and others added 14 commits January 5, 2025 01:03
Searching "fma" in the Rust documentation returns results for
`intrinsics::fma*`, but does not point to the user-facing `mul_add`. Add
aliases for `fma*` and the IEEE operation name `fusedMultiplyAdd`. Add
the IEEE name to `sqrt` as well, `squareRoot`.
This link makes sense because someone who wishes to avoid unusable `pub`
is likely, but not guaranteed, to be interested in avoiding unnameable
types.

Also fixed some grammar problems I noticed in the area.

Fixes rust-lang#116604.
…trieb

Add doc aliases for `libm` and IEEE names

Searching "fma" in the Rust documentation returns results for `intrinsics::fma*`, but does not point to the user-facing `mul_add`. Add aliases for `fma*` and the IEEE operation name `fusedMultiplyAdd`. Add the IEEE name to `sqrt` as well, `squareRoot`.
triagebot: label `src/doc/rustc-dev-guide` changes with `A-rustc-dev-guide`

Probably should also create a dev-guide reviewer pool for this repo 🤔

r? ``@Kobzol``
dev guide ping group and set adhoc reviewers to compiler

r? ``@jieyouxu`` ``@Kobzol``

I added you both to the `cc` list since you're both also on the list of owners for `src/doc/rustc-dev-guide`
Mention `unnameable_types` in `unreachable_pub` documentation.

This link makes sense because someone who wishes to avoid unusable `pub` is likely, but not guaranteed, to be interested in avoiding unnameable types.

Also fixed some grammar problems I noticed in the area.

Fixes rust-lang#116604.

r? Urgau
…=chenyukang

A few borrowck tweaks to improve 2024 edition migration lints

See first two commits' changes to test outputs. Test coverage in this area is kinda weak, but I think it affects more cases than this (like the craters that will begin to trigger the `tail_expr_drop_order` tests in rust-lang#134523).

Third commit is a drive-by change that removes a deref hack from `UseSpans` which doesn't really improve diagnostics much.
@rustbot rustbot added A-meta Area: Issues & PRs about the rust-lang/rust repository itself 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-libs Relevant to the library team, which will review and decide on the PR/issue. rollup A PR which is a rollup labels Jan 6, 2025
@matthiaskrgr
Copy link
Member Author

@bors r+ rollup=never p=6

@bors
Copy link
Collaborator

bors commented Jan 6, 2025

📌 Commit c5c05c2 has been approved by matthiaskrgr

It is now in the queue for this repository.

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

bors commented Jan 6, 2025

⌛ Testing commit c5c05c2 with merge 13738b0...

@bors
Copy link
Collaborator

bors commented Jan 6, 2025

☀️ Test successful - checks-actions
Approved by: matthiaskrgr
Pushing 13738b0 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Jan 6, 2025
@bors bors merged commit 13738b0 into rust-lang:master Jan 6, 2025
7 checks passed
@rustbot rustbot added this to the 1.86.0 milestone Jan 6, 2025
@rust-timer
Copy link
Collaborator

📌 Perf builds for each rolled up PR:

PR# Message Perf Build Sha
#135111 Add doc aliases for libm and IEEE names 51071985054ebbc0be6b11713dcc1cf6309abdc9 (link)
#135129 triagebot: label src/doc/rustc-dev-guide changes with `A-… 77f997c35deeec2f354d11a930dd2b9e8fdcecaa (link)
#135132 dev guide ping group and set adhoc reviewers to compiler 8b2cd3ae720108879d182a40751e887fb6fd1af3 (link)
#135145 Mention unnameable_types in unreachable_pub documentati… 292eea412620251c1a14e4486a19b628d05c5b94 (link)
#135147 A few borrowck tweaks to improve 2024 edition migration lin… 299d41ec52a033b6de8dcff37415f1a1f69b4847 (link)
#135150 move footnote to ordinary comment 6850cb8df48be174a9808bb5b81316b72f9138d2 (link)

previous master: fd98df8f14

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 (13738b0): 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 -4.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)
-4.1% [-4.1%, -4.1%] 1
All ❌✅ (primary) - - 0

Cycles

Results (primary 4.2%)

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)
4.2% [3.5%, 5.5%] 6
Regressions ❌
(secondary)
- - 0
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
- - 0
All ❌✅ (primary) 4.2% [3.5%, 5.5%] 6

Binary size

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

Bootstrap: 763.241s -> 764.087s (0.11%)
Artifact size: 325.66 MiB -> 325.66 MiB (0.00%)

@matthiaskrgr matthiaskrgr deleted the rollup-2vy1hwl branch January 25, 2025 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-meta Area: Issues & PRs about the rust-lang/rust repository itself 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-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants