KhronosGroup / SYCL-CTS

SYCL Conformance Tests
Apache License 2.0
61 stars 80 forks source link

[Spec change] Add sycl_khr_group_interface extension #957

Open sycl-issue-bot[bot] opened 5 days ago

sycl-issue-bot[bot] commented 5 days ago

Please review whether https://github.com/KhronosGroup/SYCL-Docs/pull/638 by @Pennycook requires any changes to the CTS.

If changes are required: Open a new PR addressing the changes and link it to this issue. If no changes are required: Close this issue and proceed with the spec PR.

Pennycook commented 2 days ago

The CTS will require additional tests:

One thing that isn't clear to me is exactly how we want to define the expected values of these tests. For the DPC++ implementation of the prototype, I compared against functionality that is already tested elsewhere (e.g., the value returned by khr::work_group::id() is checked against the value returned by group::get_group_id()), but the CTS could instead choose to compare with specific expected values.