Skip to content
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

[Bindless] Structs & enums for image types and cubemap enablement #977

Closed
wants to merge 1 commit into from

Conversation

Seanst98
Copy link
Contributor

@Seanst98 Seanst98 commented Oct 20, 2023

Add structs/enums to distinguish between image types that can be represented equivalently in memory:

  • ur_exp_image_type_desc_t struct to hold the type of an image
  • ur_exp_image_type_t enum to enable distinguishing between image types that have the same underlying memory ordering/type

Add structs/enums for cubemap sampling:

  • ur_exp_sampler_cubemap_properties_t struct to enable seamless filtering between cubemap faces
  • ur_exp_sampler_cubemap_filter_mode_t enum to enable distinguishing between cubemap seamless filtering options

This should enable image arrays, cubemaps, and combined image types.

Add structs/enums to distinguish between image types that can be represented
equivalently in memory:

 - ur_exp_image_type_desc_t struct to hold the type of an image
 - ur_exp_image_type_t enum to enable distinguishing between image types
   that have the same underlying memory ordering/type

Add structs/enums for cubemap sampling:

 - ur_exp_sampler_cubemap_properties_t struct to enable seamless filtering
   between cubemap faces
 - ur_exp_sampler_cubemap_filter_mode_t enum to enable distinguishing
   between cubemap seamless filtering options

Co-authored-by: Sean Stirling <sean.stirling@codeplay.com>
Co-authored-by: Isaac Ault <isaac.ault@codeplay.com>
Co-authored-by: Przemek Malon <przemek.malon@codeplay.com>
@Seanst98 Seanst98 changed the title [Bindless] Structs & enums for image render types and cubemap enablement [Bindless] Structs & enums for image types and cubemap enablement Oct 23, 2023
@fabiomestre fabiomestre changed the base branch from adapters to main December 5, 2023 16:53
@fabiomestre fabiomestre requested a review from a team as a code owner December 5, 2023 16:53
@fabiomestre
Copy link
Contributor

I have updated the target branch of this PR from the adapters branch to the main branch.
Development in UR is moving back to main. The adapters branch will soon be deleted.

@Seanst98 Seanst98 closed this Mar 1, 2024
@Seanst98
Copy link
Contributor Author

Seanst98 commented Mar 1, 2024

This work is outdated and will be replaced by a new PR for cubemaps

@Seanst98 Seanst98 deleted the sean/cubemaps branch March 1, 2024 10:35
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.

3 participants