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

Add: KHR_shader_integer_dot_product #2119

Merged

Conversation

spnda
Copy link
Collaborator

@spnda spnda commented Jan 7, 2024

Requires KhronosGroup/SPIRV-Cross#2257. I haven't updated the revision yet because of the outstanding PR.

Passes 100% of related CTS tests.

Also #1930 is not properly updated anymore.

Copy link
Contributor

@billhollings billhollings left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM once the SPIRV-Cross update is ready and added to this PR.

Also #1930 is not properly updated anymore.

I update that list once the PRs are merged into main. For instance, VK_KHR_format_feature_flags2 is checked now.

MoltenVK/MoltenVK/GPUObjects/MVKDevice.mm Show resolved Hide resolved
@spnda spnda force-pushed the KHR_shader_integer_dot_product branch from ec32b2e to 0833a38 Compare January 9, 2024 07:47
@spnda spnda force-pushed the KHR_shader_integer_dot_product branch from 0833a38 to 6dbe768 Compare January 16, 2024 19:40
@spnda spnda force-pushed the KHR_shader_integer_dot_product branch from 6dbe768 to 151e940 Compare January 16, 2024 19:42
@spnda
Copy link
Collaborator Author

spnda commented Jan 16, 2024

@billhollings Just updated the SPIRV-Cross ref with Hans-Kristian's additional changes, this is good to go now.

@billhollings billhollings merged commit 133a49c into KhronosGroup:main Jan 16, 2024
6 checks passed
@billhollings
Copy link
Contributor

@billhollings Just updated the SPIRV-Cross ref with Hans-Kristian's additional changes, this is good to go now.

Thanks! Merged, and updated #1930.

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