Mesa (main): broadcom/compiler: remove one superfluous call to nir_opt_undef

[GitLab Mirror]

Module: Mesa
Branch: main
Commit: 54e2e44f991a71fb77a6b100709dcf1ac3238d22
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=54e2e44f991a71fb77a6b100709dcf1ac3238d22

Author: Alejandro Piñeiro <apinhe...@igalia.com>
Date:   Mon Jan  8 14:13:13 2024 +0100

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 <ito...@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/26928>

---

 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);