-
Notifications
You must be signed in to change notification settings - Fork 787
[SYCL][HIP] Adds support for PCI device id and UUID #8233
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
@npmiller If you can also comment. Wasn't able to choose additional reviewers |
It looks the macro are related to the compute capability and the ones used for the PR were related to the driver-version
…________________________________
From: zjin-lcf ***@***.***>
Sent: Thursday, February 16, 2023 10:52 AM
To: intel/llvm ***@***.***>
Cc: Bagusetty, Abhishek ***@***.***>; Mention ***@***.***>
Subject: Re: [intel/llvm] [SYCL][HIP] Adds support for PCI device id and UUID (PR #8233)
I have a question for @abagusetty<https://github.com/abagusetty> . Are these HIP macros related to the issue ROCm/ROCm#1907<ROCm/ROCm#1907> ?
—
Reply to this email directly, view it on GitHub<#8233 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AOHFYYO7ASN4OUGGWE63DJDWXZLL3ANCNFSM6AAAAAAUUH3P7M>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
@llvm-reviewers-runtime can I get help with review and merge |
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.
Nice. :-)
Can I get some help with review and merge @intel/llvm-reviewers-runtime |
@intel/llvm-reviewers-runtime Friendly ping |
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
Completes the extensions for HIP.