From 4df93a54f18ff8baca1d7a1b395c1fa70d65a9bb Mon Sep 17 00:00:00 2001 From: Connor Abbott Date: Thu, 6 Jul 2017 12:23:33 -0700 Subject: [PATCH] nir/lower_io_to_temporaries: don't set compact on shadow vars The compact flag doesn't make sense on local variables, since the packing on them is up to the driver. This fixes nir_validate assertions in some cases, particularly when lower_io_to_temporaries is used on per-vertex inputs/outputs. Reviewed-by: Jason Ekstrand --- src/compiler/nir/nir_lower_io_to_temporaries.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/compiler/nir/nir_lower_io_to_temporaries.c b/src/compiler/nir/nir_lower_io_to_temporaries.c index 6031bbdb520..d2df14ed1e0 100644 --- a/src/compiler/nir/nir_lower_io_to_temporaries.c +++ b/src/compiler/nir/nir_lower_io_to_temporaries.c @@ -141,6 +141,7 @@ create_shadow_temp(struct lower_io_state *state, nir_variable *var) temp->data.mode = nir_var_global; temp->data.read_only = false; temp->data.fb_fetch_output = false; + temp->data.compact = false; return nvar; } -- 2.30.2