-
Notifications
You must be signed in to change notification settings - Fork 12.1k
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
Small improvements to the documentaion of FnAbi #125154
Conversation
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @wesleywiser (or someone else) some time within the next two weeks. Please see the contribution instructions for more information. Namely, in order to ensure the minimum review times lag, PR authors and assigned reviewers should ensure that the review label (
|
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix the nits and squash into one commit.
I have fixed the nits, and squashed the commits. Should be good to merge now. |
@bors r+ |
@bors rollup=always |
Rollup of 5 pull requests Successful merges: - rust-lang#125003 (avoid using aligned_alloc; posix_memalign is better-behaved) - rust-lang#125142 (Migrate `run-make/rustdoc-themes` to new rmake.rs) - rust-lang#125146 (Migrate `run-make/panic-impl-transitive` to `rmake`) - rust-lang#125154 (Small improvements to the documentaion of FnAbi ) - rust-lang#125159 (Meta: Allow unauthenticated users to modify `L-*`, `PG-*` and `-Z*` labels) r? `@ghost` `@rustbot` modify labels: rollup
Rollup merge of rust-lang#125154 - FractalFir:fnabi_doc, r=compiler-errors Small improvements to the documentaion of FnAbi I have updated the documentation of `FnAbi`. The `arg` and `ret` fields are no longer LLVM types, but Rust types(`ArgAbi` contains a `TyAndLayout` and a `PassMode`), so I changed the documentation to reflect that. Besides that, I also added documentation to other fields, and added a clarification about the differences between `FnAbi` and `FnSig`, since this is not something that is immediately obvious.
I have updated the documentation of
FnAbi
.The
arg
andret
fields are no longer LLVM types, but Rust types(ArgAbi
contains aTyAndLayout
and aPassMode
), so I changed the documentation to reflect that.Besides that, I also added documentation to other fields, and added a clarification about the differences between
FnAbi
andFnSig
, since this is not something that is immediately obvious.