From: Jason Ekstrand Date: Sat, 7 Apr 2018 05:34:57 +0000 (-0700) Subject: nir/lower_system_values: Assert/assume direct var derefs X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=aef4213fca6a8e6d0c6f3b0abeabd878e0c1d351;p=mesa.git nir/lower_system_values: Assert/assume direct var derefs System values are never arrays or structs so we can assume a direct var deref. This simplifies things a bit and prevents us from accidentally throwing away an array index. Suggested-by: Caio Marcelo de Oliveira Filho Reviewed-by: Caio Marcelo de Oliveira Filho Acked-by: Rob Clark Acked-by: Bas Nieuwenhuizen Acked-by: Dave Airlie Reviewed-by: Kenneth Graunke --- diff --git a/src/compiler/nir/nir_lower_system_values.c b/src/compiler/nir/nir_lower_system_values.c index a5f44e657a5..f315b7ae96f 100644 --- a/src/compiler/nir/nir_lower_system_values.c +++ b/src/compiler/nir/nir_lower_system_values.c @@ -41,12 +41,21 @@ convert_block(nir_block *block, nir_builder *b) if (load_deref->intrinsic != nir_intrinsic_load_deref) continue; - nir_variable *var = - nir_deref_instr_get_variable(nir_src_as_deref(load_deref->src[0])); - - if (var->data.mode != nir_var_system_value) + nir_deref_instr *deref = nir_src_as_deref(load_deref->src[0]); + if (deref->mode != nir_var_system_value) continue; + if (deref->deref_type != nir_deref_type_var) { + /* The only one system value that is an array and that is + * gl_SampleMask which is always an array of one element. + */ + assert(deref->deref_type == nir_deref_type_array); + deref = nir_deref_instr_parent(deref); + assert(deref->deref_type == nir_deref_type_var); + assert(deref->var->data.location == SYSTEM_VALUE_SAMPLE_MASK_IN); + } + nir_variable *var = deref->var; + b->cursor = nir_after_instr(&load_deref->instr); nir_ssa_def *sysval = NULL;