search

AtomLinter / linter-glsl

Atom package that lints GLSL shaders on the fly.
https://atom.io/packages/linter-glsl
MIT License
15 stars 6 forks source link

Allow GLSL for Vulkan #51

Open PopTart-NZ opened 7 years ago

PopTart-NZ commented 7 years ago

It would be great to have a way to invoke glslangvalidator such that it can parse Vulkan specific GLSL (ie. compile to SPIR-V using the -V option). Currently, lines such as layout(push_constant) uniform get flagged as an error, with the error message being 'push_constant' : only allowed when using GLSL for Vulkan. Cheers.

nyorain commented 7 years ago

I really need this feature as well. This commit (actually only 3 lines) adds full vulkan shader support, but I could not yet fully test it (might break for some non-vulkan shaders), it breaks windows support and probably results in worse performance. Best would be probably to add an option to toggle this. And if anyone here cares for windows, there probably is some way to redirect output on windows to some null/dummy file, that would need a fix as well before it can be merged i guess...

c0nd3v commented 6 years ago

Any update on this? Thanks