Skip to content

spirv-val: Add Vulkan Image Format check#6597

Merged
alan-baker merged 2 commits intoKhronosGroup:mainfrom
spencer-lunarg:spencer-lunarg-image-format-v3
Mar 24, 2026
Merged

spirv-val: Add Vulkan Image Format check#6597
alan-baker merged 2 commits intoKhronosGroup:mainfrom
spencer-lunarg:spencer-lunarg-image-format-v3

Conversation

@spencer-lunarg
Copy link
Copy Markdown
Contributor

4 years, and a 3rd try of #4748 and #5458 to get VUID-StandaloneSpirv-Image-04965 in

Last time this stalled on https://gitlab.khronos.org/spirv/SPIR-V/-/issues/766 / https://gitlab.khronos.org/spirv/SPIR-V/-/merge_requests/302/

@spencer-lunarg spencer-lunarg force-pushed the spencer-lunarg-image-format-v3 branch from db13543 to 77d7711 Compare March 19, 2026 18:21
@spencer-lunarg spencer-lunarg force-pushed the spencer-lunarg-image-format-v3 branch from 77d7711 to dbf977c Compare March 19, 2026 18:39
@spencer-lunarg
Copy link
Copy Markdown
Contributor Author

fatal error: generated SPIR-V is invalid: [VUID-StandaloneSpirv-Image-04965] Image Format type does not match Sample Type operand (float)
  %type_buffer_image_0 = OpTypeImage %float Buffer 2 0 0 1 Rgba16ui

note: please file a bug report on https://github.com/Microsoft/DirectXShaderCompiler/issues with source code if possible

@alan-baker I think this is catching a legit dxc bug?

@alan-baker
Copy link
Copy Markdown
Contributor

@alan-baker I think this is catching a legit dxc bug?

Looks like it. The glslang test failure looks legit too.

@alan-baker
Copy link
Copy Markdown
Contributor

@spencer-lunarg will you take care of the glslang update?

@spencer-lunarg
Copy link
Copy Markdown
Contributor Author

will you take care of the glslang update?

yes, what is the process, I will have time to look for a fix in Monday

@alan-baker
Copy link
Copy Markdown
Contributor

will you take care of the glslang update?

yes, what is the process, I will have time to look for a fix in Monday

I'm happy to either merge this now or wait till you have a fix. I leave it to you.

@spencer-lunarg
Copy link
Copy Markdown
Contributor Author

@alan-baker I got the glslang hot fix up KhronosGroup/glslang#4190

@spencer-lunarg
Copy link
Copy Markdown
Contributor Author

@alan-baker KhronosGroup/glslang#4191 was merged in, so GLSL is fixed now

@alan-baker alan-baker merged commit 42956b7 into KhronosGroup:main Mar 24, 2026
20 of 22 checks passed
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