Mesa (master): ac/nir: add break statements in needs_view_index_sgpr()

Samuel Pitoiset hakzsam at kemper.freedesktop.org
Thu Jan 25 13:00:13 UTC 2018


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

Author: Samuel Pitoiset <samuel.pitoiset at gmail.com>
Date:   Thu Jan 25 10:49:23 2018 +0100

ac/nir: add break statements in needs_view_index_sgpr()

Previous code is correct but as the first case statement uses
a break, keep it consistent.

CID: 1428579
Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>

---

 src/amd/common/ac_nir_to_llvm.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 8ae8650a7b..7b56edf8e7 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/common/ac_nir_to_llvm.c
@@ -555,10 +555,12 @@ static bool needs_view_index_sgpr(struct nir_to_llvm_context *ctx,
 	case MESA_SHADER_TESS_EVAL:
 		if (ctx->shader_info->info.needs_multiview_view_index || (!ctx->options->key.tes.as_es && ctx->options->key.has_multiview_view_index))
 			return true;
+		break;
 	case MESA_SHADER_GEOMETRY:
 	case MESA_SHADER_TESS_CTRL:
 		if (ctx->shader_info->info.needs_multiview_view_index)
 			return true;
+		break;
 	default:
 		break;
 	}




More information about the mesa-commit mailing list