-
Notifications
You must be signed in to change notification settings - Fork 116
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
Add DEVICE_INFO_GLOBAL_VARIABLE_SUPPORT device info query. #1145
Conversation
987f0f2
to
5303368
Compare
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #1145 +/- ##
==========================================
- Coverage 15.74% 15.72% -0.02%
==========================================
Files 223 223
Lines 31466 31494 +28
Branches 3556 3558 +2
==========================================
- Hits 4953 4952 -1
- Misses 26462 26491 +29
Partials 51 51 ☔ View full report in Codecov by Sentry. |
I have updated the target branch of this PR from the |
This gates support for the EnqueueDeviceGlobalVariableRead and Write operations. The CTS tests for these now check for support before running, and all adapters report their support correctly.
5303368
to
a0224b2
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.
Looks good for level_zero.
friendly ping @oneapi-src/unified-runtime-cuda-write @oneapi-src/unified-runtime-hip-write @npmiller |
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.
HIP and CUDA LGTM
merged into #1212 |
This gates support for the EnqueueDeviceGlobalVariableRead and Write operations. The CTS tests for these now check for support before running, and all adapters report their support correctly.
LLVM testing intel/llvm#12264