On Saturday, April 2, 2016 3:03:54 PM PDT Timothy Arceri wrote:
> We already have a count of active SSBOs per stage so use it.
> ---
> src/compiler/glsl/linker.cpp | 8 +-------
> 1 file changed, 1 insertion(+), 7 deletions(-)
>
> diff --git a/src/compiler/glsl/linker.cpp b/src/compiler/glsl/linker.cpp
> index 649fb7c..c9eaa6b 100644
> --- a/src/compiler/glsl/linker.cpp
> +++ b/src/compiler/glsl/linker.cpp
> @@ -3075,13 +3075,7 @@ check_image_resources(struct gl_context *ctx, struct
gl_shader_program *prog)
> ctx->Const.Program[i].MaxImageUniforms);
>
> total_image_units += sh->NumImages;
> -
> - for (unsigned j = 0; j < prog->NumBufferInterfaceBlocks; j++) {
> - int stage_index = prog->InterfaceBlockStageIndex[i][j];
> - if (stage_index != -1 &&
> - sh->BufferInterfaceBlocks[stage_index]->IsShaderStorage)
> - total_shader_storage_blocks++;
> - }
> + total_shader_storage_blocks += sh->NumShaderStorageBlocks;
>
> if (i == MESA_SHADER_FRAGMENT) {
> foreach_in_list(ir_instruction, node, sh->ir) {
> I think the title of this patch should be "glsl: simplify SSBO resources check", not "image"? Patches 2-3 are: Reviewed-by: Kenneth Graunke <[email protected]>
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
