Mesa (master): i965: Drop unnecessary switch statement in nir_setup_outputs()

Kenneth Graunke kwg at kemper.freedesktop.org
Mon Oct 24 21:34:24 UTC 2016


Module: Mesa
Branch: master
Commit: 3728ee000aecb19793dec56d45aff9d6cfce3e5b
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=3728ee000aecb19793dec56d45aff9d6cfce3e5b

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Wed Oct 12 22:30:30 2016 -0700

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 at lists.freedesktop.org
Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Iago Toral Quiroga <itoral at igalia.com>
Reviewed-by: Timothy Arceri <timothy.arceri at collabora.com>

---

 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 4e68ffb..9cad1a6 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(&reg, 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(&reg, var->type, &location);
    }
 }
 




More information about the mesa-commit mailing list