-
Notifications
You must be signed in to change notification settings - Fork 190
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
Reorganize PTX docs to match generator #2929
Conversation
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.
Hi, this is really great! I spot-checked the rendered docs, and it looks good.
To make reviewing easier, I undid the renaming of the file non-generated files (diff). That made the diff a lot easier to comb through.
Co-authored-by: Allard Hendriksen <[email protected]>
b2004ea
to
3cd13f0
Compare
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! For next time, I would prefer if changes in response to review could be provided as a separate commit. Especially with large changes like this were it is easy to miss something.
Co-authored-by: Allard Hendriksen <[email protected]>
Co-authored-by: Allard Hendriksen <[email protected]>
Co-authored-by: Allard Hendriksen <[email protected]>
This PR splits the
cuda::ptx
documentation into the generated and manually written part. It's just moving around content and renaming files. No other changes.