-
Notifications
You must be signed in to change notification settings - Fork 787
[UR] Document how to customize HIP directories #12127
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
Conversation
03fd673
to
d92633b
Compare
21a94c6
to
3cdbde4
Compare
Corresponding change in UR: oneapi-src/unified-runtime#1170 |
2c547d0
to
9d31976
Compare
9d31976
to
3c7275a
Compare
@kbenzie is there anything else that needs to be done for this (and oneapi-src/unified-runtime#1170) to go in? |
For this the last thing would be updating the UR tag once oneapi-src/unified-runtime#1170 is merged (I'll comment over there for next steps). I'll hold off approving this until that's done. |
2b45041
to
37bd40f
Compare
@dpcpp-doc-reviewers @ldrumm is there anything else you'd like to see done for this patch? Thank you. |
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.
@ldrumm is there anything else you'd like to see done for this patch? Thank you.
Sorry. Missed this one. Looks good modulo minor nits.
@intel/dpcpp-doc-reviewers please review, other UR changes are waiting on this. |
Co-authored-by: Kenneth Benzie (Benie) <[email protected]>
Co-authored-by: Kenneth Benzie (Benie) <[email protected]>
Co-authored-by: ldrumm <[email protected]>
1989087
to
8d0a9de
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!
No description provided.