From 63cf8adb12440512226dddbe3e233bcba87c7c18 Mon Sep 17 00:00:00 2001 From: Jason Ekstrand Date: Fri, 24 Jul 2020 10:04:48 -0500 Subject: [PATCH] spirv: Also copy over binding information for atomic counters I missed this if statement so atomic counters weren't getting bindings and, when you have more than one of them, that meant they were all getting combined into one. Fixes: 3584cb09bc15 "spirv: Give atomic counters their own variable mode" Reviewed-by: Caio Marcelo de Oliveira Filho Part-of: --- src/compiler/spirv/vtn_variables.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/compiler/spirv/vtn_variables.c b/src/compiler/spirv/vtn_variables.c index e4013e940ac..2100b481cce 100644 --- a/src/compiler/spirv/vtn_variables.c +++ b/src/compiler/spirv/vtn_variables.c @@ -2341,7 +2341,8 @@ vtn_create_variable(struct vtn_builder *b, struct vtn_value *val, if (var->mode == vtn_variable_mode_uniform || var->mode == vtn_variable_mode_ubo || - var->mode == vtn_variable_mode_ssbo) { + var->mode == vtn_variable_mode_ssbo || + var->mode == vtn_variable_mode_atomic_counter) { /* XXX: We still need the binding information in the nir_variable * for these. We should fix that. */ -- 2.30.2