-
Notifications
You must be signed in to change notification settings - Fork 788
[SYCL] Update Level Zero Loader to v1.11.0 #9979
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
This patch updates Level Zero Loader to version v1.11.0 because the previous version does not support implementation of oneapi-src/level-zero-spec#23 Level Zero extension, required for implementing sycl_ext_oneapi_device_architecture on host Addresses intel#9843 (comment)
CUDA test failed, unrelated to this patch:
|
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!
Failure in post-commit: https://github.com/intel/llvm/actions/runs/5318954472/jobs/9630923879#step:7:8914 Build failed because Level Zero loader has warnings. These warnings treated as errors. @jandres742 does the next release - v1.12.0 - has the same warnings? If yes, they block implementation of sycl_ext_oneapi_device_architecture feature, and the fix in Level Zero loader is required. If there are no warnings in v1.12.0, we can update to v1.12.0. |
thanks @dm-vodopyanov @bmyates , @nrspruit : please see the errors we are getting from the loader. I think they remain still on latest version, correct? If so, could they be fixed and made available in a new version of the loader released quick?
|
@dm-vodopyanov SYCL is enabling many more warnings in L0 loader build beyond what L0 loader CI uses. Fastest way to fix this issue would be for SYCL to imply ignore this specific warning. SYCL is already doing this for a few other warnings here:
Please also file a JIRA in GSD project and we will fix root cause in loader. |
Fixes build failure in post commit. Discussion: intel#9979 (comment)
@jandres742 @bmyates thanks for quick reply! Created PR with a w/a, and created an issue. |
Fixes build failure in post commit. Discussion: #9979 (comment)
…l Zero and OpenCL (#9843) This patch introduces new host API for sycl_ext_oneapi_device_architecture extension and implements it, currently only for Level Zero and OpenCL Depends on - oneapi-src/unified-runtime#573 - #9873 - #9979 - #10054
…l Zero and OpenCL (intel#9843) This patch introduces new host API for sycl_ext_oneapi_device_architecture extension and implements it, currently only for Level Zero and OpenCL Depends on - oneapi-src/unified-runtime#573 - intel#9873 - intel#9979 - intel#10054
…l Zero and OpenCL (#9843) This patch introduces new host API for sycl_ext_oneapi_device_architecture extension and implements it, currently only for Level Zero and OpenCL Depends on - oneapi-src#573 - intel/llvm#9873 - intel/llvm#9979 - intel/llvm#10054
This patch updates Level Zero Loader to version v1.11.0 because
the previous version does not support implementation of
oneapi-src/level-zero-spec#23 Level Zero extension, required for
implementing sycl_ext_oneapi_device_architecture on host
Addresses #9843 (comment)