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

c: allow access to gl_plain_uniforms resources #2373

Merged
merged 1 commit into from
Sep 2, 2024

Conversation

chyyran
Copy link
Contributor

@chyyran chyyran commented Aug 31, 2024

Fixes #2369

@CLAassistant
Copy link

CLAassistant commented Aug 31, 2024

CLA assistant check
All committers have signed the CLA.

@@ -226,6 +226,7 @@ typedef enum spvc_resource_type
SPVC_RESOURCE_TYPE_ACCELERATION_STRUCTURE = 12,
SPVC_RESOURCE_TYPE_RAY_QUERY = 13,
SPVC_RESOURCE_TYPE_SHADER_RECORD_BUFFER = 14,
SPVC_RESOURCE_TYPE_GL_PLAIN_UNIFORM = 15,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to bump SPVC_C_API_VERSION_MINOR, as well as spirv-cross-abi-minor in CMakeLists.txt when you add anything backwards compatible to the headers.

Copy link
Contributor Author

@chyyran chyyran Sep 2, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I bumped it to 63 for this PR, will bump it to 64 in #2374. This one should go in before #2374 to preserve ordering.

Copy link
Contributor

@HansKristian-Work HansKristian-Work left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit, otherwise fine.

@chyyran
Copy link
Contributor Author

chyyran commented Sep 2, 2024

Bumped C minor ABI version to 63 in b2171de

@HansKristian-Work HansKristian-Work merged commit c1161c5 into KhronosGroup:main Sep 2, 2024
6 checks passed
@chyyran chyyran deleted the c_gl_plain_uniforms branch October 8, 2024 06:16
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.

gl_plain_uniforms resources can not be access via C API
3 participants