-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Document ABI registers #25123
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
Document ABI registers #25123
Conversation
Thanks! This might be more discoverable if we put it in a doc under |
NP. Having it close to discussion of the context/error regs seemed the most discoverable to me. |
Maybe that whole section of the document could be extracted into a new document(s) under the |
Up to you, I confess I looked in the ABI directory when I was trying to puzzle this out. |
dfaad6e
to
24b1f97
Compare
@rjmccall or @aschwaighofer should probably look over this for accuracy too. |
"Register allocation" is an unfortunate name for this document. Maybe it could just be an appendix in That file needs some work, too. Among other things, it needs to be converted to Markdown. |
@rjmccall I think we should migrate it into subdocuments under the |
By "it" you mean I suppose we should make a decision about whether target-specific information should be split into appendices of a bunch of different files or unified in a single file per target. The disadvantage of the former is that we're more likely to forget to include information about how a particular target implements a particular ABI feature; the disadvantage of the latter is that the different targets are likely to diverge in terms of the information they include in their documents and how they present it. |
Looking at it now, I think it's fine as a single document. I think I had |
Co-Authored-By: Xiaodi Wu <[email protected]>
Co-Authored-By: Xiaodi Wu <[email protected]>
Co-Authored-By: Xiaodi Wu <[email protected]>
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.
LGTM; any other comments @rjmccall @stephentyrone ?
@swift-ci Please smoke test |
This is fine; I'd still like to reach agreement about the point I raised above, but we don't need to pretend that this commit will be the last word. |
Thanks everyone 👍 |
Hi Apple,
A minor change to ABI manifesto to document register usage in the Swift ABI as it doesn’t seem to be documented elsewhere. If it is, just let me know!
Thanks,
John