nir/lower_viewport: Check variable mode first

The location is unused for shader_temp and function_temp variables, and
due to the way we nir_lower_io_to_temproraries demotes shader_out
variables to shader_temp variables, it happened to equal
VARYING_SLOT_POS for the gl_Position temporary, which made this pass
fail with the offline compiler due to this coming before vars_to_ssa.

Reviewed-by: Qiang Yu <yuq825@gmail.com>
This commit is contained in:
Connor Abbott
2019-06-10 21:59:12 +02:00
parent 6e5e4bf050
commit 4423552ff0

View File

@@ -56,7 +56,8 @@ nir_lower_viewport_transform(nir_shader *shader)
continue;
nir_variable *var = nir_intrinsic_get_var(intr, 0);
if (var->data.location != VARYING_SLOT_POS)
if (var->data.mode != nir_var_shader_out ||
var->data.location != VARYING_SLOT_POS)
continue;
nir_builder b;