-
Notifications
You must be signed in to change notification settings - Fork 192
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
glslang compilation error when using uvec2 in GLSL code #1077
Comments
The VSG builds glslang as part of the VSG itself to compile GLSL shaders to SPIR-V, so either the version of glslang we are including has a problem, or glslang requires an option enabled to support uvec's. Could you try out glslangValidator on the shader? |
Yesterday evening I tried out uvec2 with shaders on my laptop and reproduced the issue, I tried glslangValidator from VulkanSDK 1.2.268 and 1.3.275 and both acceoted my uvec2 code, I then checked the glslang version used in each case. The VSG's glslang is 12.0.0 which was drawn from glslang master last year, while the VulkanSDK versions were 10.x. and 11.x versions respectively, so it could be there was a regression between 11.x and 12.x. The VSG uses a branch from glslang so I there is changes the upstream glslang has resolved this regression and we just need to merge updates. I have other tasks to tackle to today so can't spend time right away investigating this further. |
@asafran I have created a branch of the VSG that links to glslang as an external dependency rather than then building a fork of glslang directly into VSG library. I've written up some details on the discussion therad #1199. Could you test of the new external_glslang branch with modern version of gslang (14.1 or later) to see if the issue uvec2 issue is resolved. Thanks. |
Describe the bug
glslang compilation error when using uvec2 in GLSL code
To Reproduce
Steps to reproduce the behavior:
const uvec2 range = uvec2(2, 5);
Expected behavior
Successful compilation of the GLSL code
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: