-
Notifications
You must be signed in to change notification settings - Fork 78
[1.3] Gitlab CI: enable test for the SYCL CPU backend #2462
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
[1.3] Gitlab CI: enable test for the SYCL CPU backend #2462
Conversation
Mhm, looks like we cannot enable this for the 1.3.0 branch then. |
That's strange. Before we could compile it with ICPX 2024.2 and C++17. If I'm not wrong, you only changed the OneAPI version and did not install the Open-CL Runtime anymore. |
@fwyzard I spent some time with the issue. The update to OneAPI 2025 causes the problem. The new compiler throws this warning, which becomes an error because of our build configuration. There is already a catch2 issue for it: catchorg/Catch2#2910 |
I see. I would propose to suppress the warning, until the fix is merged in CMake or if we use a version without the fix. |
I agree. It's working with warning and the warning will be gone, when we move to C++20. I will work on a CMake fix for our alpaka CMake. |
@fwyzard Can you please apply the patch on your branch. This suppress the warning and should fix the problem. |
@SimeonEhrig thank you for the patch. Do you know if clang 19 would have the same issue ? |
|
@fwyzard I extended the patch for clang 19. You need to revert the patch. |
7bf0884
to
51077df
Compare
@SimeonEhrig thank you for the update. |
@fwyzard Now trigger the wrap-test bug also here: #2464 (comment) |
Yes... unfortunately that looks to be a real issue: #2464 (comment) . |
I merge after #2469 is merged. |
2d0d555
to
18ef48a
Compare
(rebased to fix the conflicts) |
No description provided.