Skip to content

[SYCL][Bindless][Exp] Allow read_image/mipmap to return user types #11961

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
merged 1 commit into from
Nov 21, 2023

Conversation

przemektmalon
Copy link
Contributor

  • read_image and read_mipmap now allow user-defined types to be returned.

  • The write_image test case for user-defined types is updated to include both reading and writing of user-defined types.

  • An additional test is added for reading user-defined types where the image type is a mipmap.

  • The extension document has been updated to reflect the above changes.

  • The write_image function now checks whether a user-defined type was passed to it, and converts the input data accordingly. If the type passed has a specific NVPTX backend intrinsic, it will use that, otherwise it will write the user data type bit-cast to a variant of unsigned integer.

  • This patch marks the bump to revision 5.0 of the bindless images specification document.

 - `read_image` and `read_mipmap` now allow user-defined types to be
   returned.

 - The `write_image` test case for user-defined types is updated to
   include both reading and writing of user-defined types.

 - An additional test is added for reading user-defined types where the
   image type is a mipmap.

 - The extension document has been updated to reflect the above changes.

 - The `write_image` function now checks whether a user-defined type was
   passed to it, and converts the input data accordingly. If the type
   passed has a specific NVPTX backend intrinsic, it will use that,
   otherwise it will write the user data type bit-cast to a variant of
   unsigned integer.

 - This patch marks the bump to revision `5.0` of the bindless images
   specification document.
@przemektmalon przemektmalon requested a review from a team as a code owner November 21, 2023 13:30
@againull againull changed the title [Bindless][Exp] Allow read_image/mipmap to return user types [SYCL][Bindless][Exp] Allow read_image/mipmap to return user types Nov 21, 2023
@againull againull merged commit 5c84218 into intel:sycl Nov 21, 2023
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.

4 participants