From 767c1eb43686ef6bebf7ebc99c4cef9c6f58d4af Mon Sep 17 00:00:00 2001 From: Brian Paul Date: Tue, 14 Nov 2017 09:34:29 -0700 Subject: [PATCH] svga: simplify array test in svga_init_shader_key_common() And squash commit a patch to silence a compiler warning (add default case to the switch statement). Reviewed-by: Charmaine Lee --- src/gallium/drivers/svga/svga_shader.c | 21 ++++++++++----------- 1 file changed, 10 insertions(+), 11 deletions(-) diff --git a/src/gallium/drivers/svga/svga_shader.c b/src/gallium/drivers/svga/svga_shader.c index e16842f2358..ebf1131d51d 100644 --- a/src/gallium/drivers/svga/svga_shader.c +++ b/src/gallium/drivers/svga/svga_shader.c @@ -209,18 +209,17 @@ svga_init_shader_key_common(const struct svga_context *svga, assert(view->texture->target < (1 << 4)); /* texture_target:4 */ /* 1D/2D array textures with one slice are treated as non-arrays - * by the SVGA3D device. Convert the texture type here so that - * we emit the right TEX/SAMPLE instruction in the shader. + * by the SVGA3D device. Set the is_array flag only if we know that + * we have more than 1 element. This will be used to select shader + * instruction/resource types during shader translation. */ - if (view->texture->target == PIPE_TEXTURE_1D_ARRAY || - view->texture->target == PIPE_TEXTURE_2D_ARRAY) { - if (view->texture->array_size == 1) { - key->tex[i].is_array = 0; - } - else { - assert(view->texture->array_size > 1); - key->tex[i].is_array = 1; - } + switch (view->texture->target) { + case PIPE_TEXTURE_1D_ARRAY: + case PIPE_TEXTURE_2D_ARRAY: + key->tex[i].is_array = view->texture->array_size > 1; + break; + default: + ; /* nothing / silence compiler warning */ } assert(view->texture->nr_samples < (1 << 5)); /* 5-bit field */ -- 2.30.2