From a19dc3bf00bb031120805b1e40fe0a3a455648b7 Mon Sep 17 00:00:00 2001
From: Levi Behunin <l3ehunin@gmail.com>
Date: Thu, 24 Sep 2020 21:52:23 -0600
Subject: Address Comments

---
 src/video_core/shader/shader_ir.cpp | 49 ++++++++++++++++++++-----------------
 1 file changed, 27 insertions(+), 22 deletions(-)

(limited to 'src/video_core/shader/shader_ir.cpp')

diff --git a/src/video_core/shader/shader_ir.cpp b/src/video_core/shader/shader_ir.cpp
index 14206d3ae..521d0e47d 100644
--- a/src/video_core/shader/shader_ir.cpp
+++ b/src/video_core/shader/shader_ir.cpp
@@ -388,12 +388,12 @@ void ShaderIR::SetInternalFlagsFromInteger(NodeBlock& bb, Node value, bool sets_
         return;
     }
     switch (value->index()) {
-    case 0:
+    case 0: // Operation Node
         Iterop(bb, value);
         break;
-    case 2:
+    case 2: // Genral Purpose Node
         if (const auto gpr = std::get_if<GprNode>(value.get())) {
-            LOG_WARNING(HW_GPU, "GprNode: index={}", gpr->GetIndex());
+            LOG_DEBUG(HW_GPU, "GprNode: index={}", gpr->GetIndex());
             Node zerop = Operation(OperationCode::LogicalIEqual, std::move(value),
                                    Immediate(gpr->GetIndex()));
             SetInternalFlag(bb, InternalFlag::Zero, std::move(zerop));
@@ -408,26 +408,31 @@ void ShaderIR::SetInternalFlagsFromInteger(NodeBlock& bb, Node value, bool sets_
     }
 }
 
-void ShaderIR::Iterop(NodeBlock& nb, Node var) {
-    if (const auto op = std::get_if<OperationNode>(var.get())) {
-        if (op->GetOperandsCount() > 0) {
-            for (auto& opss : op->GetOperands()) {
-                switch (opss->index()) {
-                case 0:
-                    return Iterop(nb, opss);
-                case 2:
-                    if (const auto gpr = std::get_if<GprNode>(opss.get())) {
-                        LOG_WARNING(HW_GPU, "Child GprNode: index={}", gpr->GetIndex());
-                        Node zerop = Operation(OperationCode::LogicalIEqual, std::move(opss),
-                                               Immediate(gpr->GetIndex()));
-                        SetInternalFlag(nb, InternalFlag::Zero, std::move(zerop));
-                    }
-                    break;
-                default:
-                    LOG_WARNING(HW_GPU, "Child Node Type: {}", opss->index());
-                    break;
-                }
+void ShaderIR::SearchOperands(NodeBlock& nb, Node var) {
+    const auto* op = std::get_if<OperationNode>(var.get());
+    if (op == nullptr) {
+        return;
+    }
+
+    if (op->GetOperandsCount() == 0) {
+        return;
+    }
+
+    for (auto& operand : op->GetOperands()) {
+        switch (operand->index()) {
+        case 0: // Operation Node
+            return Iterop(nb, operand);
+        case 2: // General Purpose Node
+            if (const auto* gpr = std::get_if<GprNode>(operand.get())) {
+                LOG_DEBUG(HW_GPU, "Child GprNode: index={}", gpr->GetIndex());
+                Node zerop = Operation(OperationCode::LogicalIEqual, std::move(operand),
+                                       Immediate(gpr->GetIndex()));
+                SetInternalFlag(nb, InternalFlag::Zero, std::move(zerop));
             }
+            break;
+        default:
+            LOG_WARNING(HW_GPU, "Child Node Type: {}", operand->index());
+            break;
         }
     }
 }
-- 
cgit v1.2.3