Skip to content

Test with SYCL in the CI - #235

Open
masterleinad wants to merge 4 commits into
kokkos:stablefrom
masterleinad:test_sycl_ci
Open

Test with SYCL in the CI#235
masterleinad wants to merge 4 commits into
kokkos:stablefrom
masterleinad:test_sycl_ci

Conversation

@masterleinad

@masterleinad masterleinad commented Jan 30, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor

Based on top of #234. This pull request makes the icpx test with SYCL.
I also added --output-on-failure to CTest and needed to allow CL/sycl.hpp instead of sycl/sycl.hpp for older oneAPI releases. The header checks needs C++17 but SYCL needs C++17 support anyway.

@masterleinad
masterleinad marked this pull request as ready for review January 30, 2023 20:58
keryell
keryell reviewed May 17, 2023
View reviewed changes
Comment thread tests/offload_utils.hpp Outdated
@masterleinad
masterleinad force-pushed the test_sycl_ci branch 4 times, most recently from ef1419a to 0b9a525 Compare October 6, 2025 13:41
@tpadioleau tpadioleau mentioned this pull request Oct 27, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nmm0 nmm0 nmm0 approved these changes
+1 more reviewer
@keryell keryell keryell left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@masterleinad @keryell @nmm0