On Sunday, April 3, 2016 9:16:28 PM PDT Timothy Arceri wrote:
[snip]
> @@ -3295,8 +3296,8 @@ should_add_buffer_variable(struct gl_shader_program
*shProg,
> if (type != GL_BUFFER_VARIABLE)
> return true;
>
> - for (unsigned i = 0; i < shProg->NumBufferInterfaceBlocks; i++) {
> - const char *block_name = shProg->BufferInterfaceBlocks[i].Name;
> + for (unsigned i = 0; i < shProg->NumShaderStorageBlocks; i++) {
> + const char *block_name = shProg->ShaderStorageBlocks[i].Name;
> block_name_len = strlen(block_name);It looks like should_add_buffer_variable() is used for both uniform blocks and shader storage blocks. I don't think this is correct. The other hunks look fine.
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
