Skip to content

[SYCL] Report false for aspect::image on all devices #9368

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

Merged

Conversation

steffenlarsen
Copy link
Contributor

With the new ext_intel_legacy_image aspect for checking SYCL 1.2.1 image support, the implementation should now refer to the support for SYCL 2020 images when queried for either info::device::image_support or aspect::image. Since there is no current support for device-side SYCL 2020 images, this forces all devices to report false for both of these queries.

With the new ext_intel_legacy_image aspect for checking SYCL 1.2.1 image
support, the implementation should now refer to the support for SYCL
2020 images when queried for either info::device::image_support or
aspect::image. Since there is no current support for device-side SYCL
2020 images, this forces all devices to report false for both of these
queries.

Signed-off-by: Larsen, Steffen <[email protected]>
@steffenlarsen steffenlarsen requested a review from a team as a code owner May 9, 2023 11:16
@steffenlarsen steffenlarsen temporarily deployed to aws May 9, 2023 13:09 — with GitHub Actions Inactive
Signed-off-by: Larsen, Steffen <[email protected]>
@steffenlarsen steffenlarsen changed the title [SYCL] Report false for aspect::image on all devices [SYCL][ABI-break] Report false for aspect::image on all devices May 9, 2023
@steffenlarsen
Copy link
Contributor Author

steffenlarsen commented May 9, 2023

This is an ABI break and has now been marked and handled as such.

Correction; missing symbol was a required specialization due to it being in the __SYCL_PARAM_TRAITS_SPEC_SPECIALIZED category now. ABI-break is not needed.

@steffenlarsen steffenlarsen temporarily deployed to aws May 9, 2023 20:16 — with GitHub Actions Inactive
@steffenlarsen steffenlarsen temporarily deployed to aws May 9, 2023 22:47 — with GitHub Actions Inactive
@steffenlarsen steffenlarsen changed the title [SYCL][ABI-break] Report false for aspect::image on all devices [SYCL] Report false for aspect::image on all devices May 10, 2023
@steffenlarsen steffenlarsen temporarily deployed to aws May 11, 2023 16:33 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants