From d7ea433a5fc13ef1f622fddf01afb1f1de7bc6b4 Mon Sep 17 00:00:00 2001 From: Caio Marcelo de Oliveira Filho Date: Wed, 19 Jun 2019 10:00:39 -0700 Subject: [PATCH] glsl/nir: Fix getting the sampler dim when arrays are involved Unwrap any array in the variable type so we can get the sampler dim. This fixes piglit test spec/arb_arrays_of_arrays/execution/image_store/basic-imageStore-const-uniform-index.shader_test. Fixes: f2d0e48ddc7 "glsl/nir: Add optimization pass for access flags" Reviewed-by: Kenneth Graunke --- src/compiler/glsl/gl_nir_opt_access.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/compiler/glsl/gl_nir_opt_access.c b/src/compiler/glsl/gl_nir_opt_access.c index 1affeb3..7f8672f 100644 --- a/src/compiler/glsl/gl_nir_opt_access.c +++ b/src/compiler/glsl/gl_nir_opt_access.c @@ -70,7 +70,8 @@ gather_intrinsic(struct access_state *state, nir_intrinsic_instr *instr) * image types use textures which cannot alias with buffer objects. * Therefore we have to group buffer samplers together with SSBO's. */ - if (glsl_get_sampler_dim(var->type) == GLSL_SAMPLER_DIM_BUF) + if (glsl_get_sampler_dim(glsl_without_array(var->type)) == + GLSL_SAMPLER_DIM_BUF) state->buffers_written = true; else state->images_written = true; -- 2.7.4