From: Kenneth Graunke Date: Thu, 5 Jul 2018 21:34:27 +0000 (-0700) Subject: nir: Avoid splitting compact arrays into per-element variables. X-Git-Tag: upstream/19.3.0~10020 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=15c69021176395a08febde51ce14a43a15fee07d;p=platform%2Fupstream%2Fmesa.git nir: Avoid splitting compact arrays into per-element variables. Compact arrays are used for special variables like clip and cull distances, or tessellation levels. Drivers using compact arrays assume that these values will always be actual arrays. We don't want to turn a float[1] gl_CullDistance into a single float; that would confuse drivers. Today, i965 uses compact arrays, and Gallium drivers use nir_lower_io_arrays_to_elements, so we haven't had any overlap that would demonstrate the issue. Iris will use both. Reviewed-by: Timothy Arceri --- diff --git a/src/compiler/nir/nir_lower_io_arrays_to_elements.c b/src/compiler/nir/nir_lower_io_arrays_to_elements.c index 73b6810..5fbde08 100644 --- a/src/compiler/nir/nir_lower_io_arrays_to_elements.c +++ b/src/compiler/nir/nir_lower_io_arrays_to_elements.c @@ -291,6 +291,10 @@ lower_io_arrays_to_elements(nir_shader *shader, nir_variable_mode mask, nir_variable *var = nir_deref_instr_get_variable(deref); + /* Drivers assume compact arrays are, in fact, arrays. */ + if (var->data.compact) + continue; + /* Skip indirects */ uint64_t loc_mask = ((uint64_t)1) << var->data.location; if (var->data.patch) {