Skip to content

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Aug 6, 2025

This fixes a few intrinsic docs that had a link directly to itself instead of to the correct function in the mem module.

This fixes a few intrinsic docs that had a link directly to itself
instead of to the correct function in the `mem` module.
@rustbot
Copy link
Collaborator

rustbot commented Aug 6, 2025

r? @scottmcm

rustbot has assigned @scottmcm.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Aug 6, 2025
@rustbot
Copy link
Collaborator

rustbot commented Aug 6, 2025

Some changes occurred to the intrinsics. Make sure the CTFE / Miri interpreter
gets adapted for the changes, if necessary.

cc @rust-lang/miri, @RalfJung, @oli-obk, @lcnr

@kpreid
Copy link
Contributor

kpreid commented Aug 6, 2025

For historical context: this error was introduced by 988eb19 “library: Use size_of from the prelude instead of imported”.

@scottmcm
Copy link
Member

scottmcm commented Aug 6, 2025

Ah, that makes sense. Thanks for fixing and spelunking!

@bors r+ rollup=always

@bors
Copy link
Collaborator

bors commented Aug 6, 2025

📌 Commit 2cd5b49 has been approved by scottmcm

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 Aug 6, 2025
bors added a commit that referenced this pull request Aug 6, 2025
Rollup of 12 pull requests

Successful merges:

 - #144552 (Rehome 33 `tests/ui/issues/` tests to other subdirectories under `tests/ui/`)
 - #144676 (Add documentation for unstable_feature_bound)
 - #144836 (Change visibility of Args new function)
 - #144910 (Add regression tests for seemingly fixed issues)
 - #144913 ([rustdoc] Fix wrong `i` tooltip icon)
 - #144924 (compiletest: add hint for when a ui test produces no errors)
 - #144926 (Correct the use of `must_use` on btree::IterMut)
 - #144928 (Drop `rust-version` from `rustc_thread_pool`)
 - #144945 (Autolabel PRs that change explicit tail call tests as `F-explicit_tail_calls`)
 - #144954 (run-make: Allow blessing snapshot files that don't exist yet)
 - #144971 (num: Rename `isolate_most_least_significant_one` functions)
 - #144978 (Fix some doc links for intrinsics)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 66f0f27 into rust-lang:master Aug 6, 2025
10 checks passed
rust-timer added a commit that referenced this pull request Aug 6, 2025
Rollup merge of #144978 - ehuss:intrinsic-doc-links, r=scottmcm

Fix some doc links for intrinsics

This fixes a few intrinsic docs that had a link directly to itself instead of to the correct function in the `mem` module.
@rustbot rustbot added this to the 1.91.0 milestone Aug 6, 2025
github-actions bot pushed a commit to rust-lang/rustc-dev-guide that referenced this pull request Aug 7, 2025
Rollup of 12 pull requests

Successful merges:

 - rust-lang/rust#144552 (Rehome 33 `tests/ui/issues/` tests to other subdirectories under `tests/ui/`)
 - rust-lang/rust#144676 (Add documentation for unstable_feature_bound)
 - rust-lang/rust#144836 (Change visibility of Args new function)
 - rust-lang/rust#144910 (Add regression tests for seemingly fixed issues)
 - rust-lang/rust#144913 ([rustdoc] Fix wrong `i` tooltip icon)
 - rust-lang/rust#144924 (compiletest: add hint for when a ui test produces no errors)
 - rust-lang/rust#144926 (Correct the use of `must_use` on btree::IterMut)
 - rust-lang/rust#144928 (Drop `rust-version` from `rustc_thread_pool`)
 - rust-lang/rust#144945 (Autolabel PRs that change explicit tail call tests as `F-explicit_tail_calls`)
 - rust-lang/rust#144954 (run-make: Allow blessing snapshot files that don't exist yet)
 - rust-lang/rust#144971 (num: Rename `isolate_most_least_significant_one` functions)
 - rust-lang/rust#144978 (Fix some doc links for intrinsics)

r? `@ghost`
`@rustbot` modify labels: rollup
github-actions bot pushed a commit to model-checking/verify-rust-std that referenced this pull request Aug 8, 2025
Fix some doc links for intrinsics

This fixes a few intrinsic docs that had a link directly to itself instead of to the correct function in the `mem` module.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. 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.

5 participants