From 54e2e44f991a71fb77a6b100709dcf1ac3238d22 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alejandro=20Pi=C3=B1eiro?= Date: Mon, 8 Jan 2024 14:13:13 +0100 Subject: [PATCH] broadcom/compiler: remove one superfluous call to nir_opt_undef v3d_optimize_nir is calling nir_opt_undef twice. As it is inside the usual "do {..} while (progress);" loop, is not needed to call it twice. Reviewed-by: Iago Toral Quiroga Part-of: --- src/broadcom/compiler/nir_to_vir.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/broadcom/compiler/nir_to_vir.c b/src/broadcom/compiler/nir_to_vir.c index 4fdcaac73c2..782d015592b 100644 --- a/src/broadcom/compiler/nir_to_vir.c +++ b/src/broadcom/compiler/nir_to_vir.c @@ -2105,7 +2105,6 @@ v3d_optimize_nir(struct v3d_compile *c, struct nir_shader *s) NIR_PASS(progress, s, nir_opt_remove_phis); NIR_PASS(progress, s, nir_opt_if, false); - NIR_PASS(progress, s, nir_opt_undef); if (c && !c->disable_gcm) { bool local_progress = false; NIR_PASS(local_progress, s, nir_opt_gcm, false);