SPIRV+OpenGL plain matrix uniform types are wrong

[cubeleo]

We are trying to migrate our project from runtime compiled shaders to precompiled SPIRV. It was going well until I tried an AMD GPU (Windows 11, Radeon RX 6800S). I'm using plain uniforms for my transformation matrices. When I use glGetActiveUniform to get their type on NVidia, I get GL_FLOAT_MAT4. But when I do it on AMD, I get GL_FLOAT_VEC4 of size 4.

I don't want to have to make weird assumptions about mat4 being an array of vec4 only on certain systems. I also don't want to just move all mat4s to arrays of vec4, since this means I have to do extra work with uniform layout locations in the shaders. I could switch to some alternative way to pass my matrices in like SSBO, but I'd rather not yet, and this seems like basic usage that should work fine (and does on other devices). I'm assuming I goofed something up and not a driver bug, since it's such a basic problem. I created a minimal repro case and pushed it to a git repo. What might cause this? FYI, this repo assumes glslangValidator is available on the dev machine.

https://github.com/cubeleo/GLSPIRVMatrixBugAMD.git

[lubaozho]

Thanks for reporting it, an internal bug ticket has been created to track this issue, will keep you posted on its progress. Thanks.

[DonglinWei2018]

This issue is understood and fixed internally. The future Adrenalin driver releases will include the complete fix.

[cubeleo]

Great to see this is fixed, thank you! Any idea when the publically available drivers will contain the fix?

[lubaozho]

Great to see this is fixed, thank you! Any idea when the publically available drivers will contain the fix?

@cubeleo Could you try the latest driver "24.8.1 Adrenalin release"?

[cubeleo]

I updated to 24.8.1 and built and ran the program in the repo I linked to in my bug report, and still get GL_FLOAT_VEC4.

[lubaozho]

I updated to 24.8.1 and built and ran the program in the repo I linked to in my bug report, and still get GL_FLOAT_VEC4.

Hi, @cubeleo the fix is not yet included into the Adrenalin driver releases, I will notify you if I get any update on this. Thanks.

[cubeleo]

Ok thanks! I have a few workarounds.