-
Notifications
You must be signed in to change notification settings - Fork 790
[SYCL][UR] Update unified-runtime #8315
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
SYCL / Linux / Build + LIT (pull_request_target) https://github.com/intel/llvm/actions/runs/4154130558/jobs/7187700934 failing with: _deps/unified-runtime-src/source/loader/ur_loader.cpp:36:2: error: extra ';' [-Werror=pedantic]
36 | };
| ^
_deps/unified-runtime-src/source/loader/ur_loader.cpp:43:2: error: extra ';' [-Werror=pedantic]
43 | }; Posted PR in UR oneapi-src/unified-runtime#254 |
I have identified some issues in tests. Fixing them in my machine and will repost. |
@smaslov-intel : do you see why this is failing? https://github.com/intel/llvm/actions/runs/4191948209/jobs/7266948415 All I see is are errors like: rm: cannot remove '/__w/llvm/llvm/bolt/test/shared-object.test': Permission denied |
I don't see either. Did you try to restart it? |
No. |
@intel/dpcpp-devops-reviewers, could you investigate, please? To reduce the impact on CI jobs we probably should disable broken systems in GitHub Actions. |
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.
Just a small nitpick, otherwise changes look good to me.
Signed-off-by: Jaime Arteaga <[email protected]>
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
Update to oneapi-src/unified-runtime@6fb8e26 Tag: weekly-2023-02-24 Signed-off-by: Jaime Arteaga <[email protected]>
Update to
oneapi-src/unified-runtime@6fb8e26
Tag: weekly-2023-02-24
Signed-off-by: Jaime Arteaga [email protected]