From 3728ee000aecb19793dec56d45aff9d6cfce3e5b Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Wed, 12 Oct 2016 22:30:30 -0700 Subject: [PATCH] i965: Drop unnecessary switch statement in nir_setup_outputs() TCS and FS are skipped above. CS has no output variables. All remaining cases take the same path. Cc: mesa-stable@lists.freedesktop.org Signed-off-by: Kenneth Graunke Reviewed-by: Iago Toral Quiroga Reviewed-by: Timothy Arceri --- src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 15 +++------------ 1 file changed, 3 insertions(+), 12 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp index 4e68ffbf05b..9cad1a6d295 100644 --- a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp @@ -87,18 +87,9 @@ fs_visitor::nir_setup_outputs() nir_outputs = bld.vgrf(BRW_REGISTER_TYPE_F, nir->num_outputs); nir_foreach_variable(var, &nir->outputs) { - switch (stage) { - case MESA_SHADER_VERTEX: - case MESA_SHADER_TESS_EVAL: - case MESA_SHADER_GEOMETRY: { - fs_reg reg = offset(nir_outputs, bld, var->data.driver_location); - unsigned location = var->data.location; - nir_setup_single_output_varying(®, var->type, &location); - break; - } - default: - unreachable("unhandled shader stage"); - } + fs_reg reg = offset(nir_outputs, bld, var->data.driver_location); + unsigned location = var->data.location; + nir_setup_single_output_varying(®, var->type, &location); } } -- 2.30.2