Mesa (main): glsl: Don't split arrays in the condition of an assignment

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Fri Feb 11 17:59:12 UTC 2022


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

Author: Ian Romanick <ian.d.romanick at intel.com>
Date:   Fri Jan 14 18:39:15 2022 -0800

glsl: Don't split arrays in the condition of an assignment

At this point, this should always be NULL.

v2: Fix bad rebase.

Reviewed-by: Matt Turner <mattst88 at gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14573>

---

 src/compiler/glsl/opt_array_splitting.cpp | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/src/compiler/glsl/opt_array_splitting.cpp b/src/compiler/glsl/opt_array_splitting.cpp
index 7e147ca89e3..63b1a01290c 100644
--- a/src/compiler/glsl/opt_array_splitting.cpp
+++ b/src/compiler/glsl/opt_array_splitting.cpp
@@ -435,11 +435,6 @@ ir_array_splitting_visitor::visit_leave(ir_assignment *ir)
    handle_rvalue(&ir->rhs);
    ir->rhs->accept(this);
 
-   if (ir->condition) {
-      handle_rvalue(&ir->condition);
-      ir->condition->accept(this);
-   }
-
    return visit_continue;
 }
 



More information about the mesa-commit mailing list