KhronosGroup / glslang

Khronos-reference front end for GLSL/ESSL, partial front end for HLSL, and a SPIR-V generator.
Other
2.91k stars 816 forks source link

local_size_variable #2491

Open Scenomics opened 3 years ago

Scenomics commented 3 years ago

The _local_sizevariable option is made possible by the extension _GL_ARB_compute_variable_groupsize.

https://www.khronos.org/registry/OpenGL/extensions/ARB/ARB_compute_variable_group_size.txt

Is there any plan to add support? For example being able to write in the GLSL:

version 430

layout( local_size_variable ) in; void main() {}

Where the values of _local_sizex, _local_sizey, and _local_sizez are set when compute is dispatched. Instead of having to write:

version 430

layout( local_size_x = 32, local_size_y = 32, local_size_z = 1 ) in; void main() {}

If support is not planned, are you interested in someone else doing the implementation and submitting a PR?

greg-lunarg commented 3 years ago

I am not aware of any plans. I have searched the issue database, the PR history and source.

I presume the community would like this implemented. If so, I would welcome such a PR.

I am going to ping the GLSL spec editor @gnl21 for confirmation.

Scenomics commented 3 years ago

Thank you very much for getting back to me. Now that I am able to proceed, I'll schedule the work and have more details in a few weeks.

gnl21 commented 3 years ago

I am going to ping the GLSL spec editor @gnl21 for confirmation.

I don't have any official confirmation to give, but I agree that this would be desirable and I don't know of any current efforts to add it.

Cewein commented 1 year ago

any news about this ? I could start to look into and share any progress about it if nothing as been done about it.

arcady-lunarg commented 1 year ago

Sounds like there isn't any news. @Cewein if you want look into it, you should go for it.