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(®, 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);
}
}
More information about the mesa-commit
mailing list