-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Add documentation to compiler intrinsics #68682
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Conversation
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
54a18ad
to
5380bc4
Compare
@rustbot modify labels: +T-doc |
☔ The latest upstream changes (presumably #68861) made this pull request unmergeable. Please resolve the merge conflicts. |
e02bbd7
to
c9ac80c
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
c9ac80c
to
b22d370
Compare
Ping from triage: @LeSeulArtichaut - if you've addressed the changes from steveklabnik, can you please mark it as completed so it can be reviewed? |
@JohnCSimon I thought I already marked it as resolved... I can't find a way to mark the entire review as completed, have I missed something? |
@bors r=steveklabnik rollup |
📌 Commit b22d370 has been approved by |
Rollup of 6 pull requests Successful merges: - #68682 (Add documentation to compiler intrinsics) - #69544 (Unrevert "Remove `checked_add` in `Layout::repeat`") - #69617 (constify mem::forget) - #69622 (Rename `syntax` in librustc_ast/README.md) - #69623 (stash API: remove panic to fix ICE.) - #69624 (Toolstate: Don't block beta week on already broken tools.) Failed merges: - #69626 (Toolstate: don't duplicate nightly tool list.) r? @ghost
This adds documentation to the compiler intrinsics having stable standard implementations.
Relates to #34338 (cc @bstrie)
r? @steveklabnik (for reassignment?)