lima/gpir: Make lima_gpir_node_insert_child() useful

We weren't using this function before. The name is confusing, but it
changes the child while also fixing up the dependence link, if you don't
have access to it already. Or at least, I think that's what the
intention is, and what we'll need to change the branch condition in the
next commit. Adding a dependency between the new and old source doesn't
make any sense for this, and we also need to change the actual source.

Reviewed-by: Vasily Khoruzhick <anarsoul@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4125>

diff --git a/src/gallium/drivers/lima/ir/gp/node.c b/src/gallium/drivers/lima/ir/gp/node.c
index 78f7bd1..ef534e7 100644 (file)
--- a/src/gallium/drivers/lima/ir/gp/node.c
+++ b/src/gallium/drivers/lima/ir/gp/node.c
@@ -413,10 +413,10 @@ void gpir_node_insert_child(gpir_node *parent, gpir_node *child,
    gpir_node_foreach_pred(parent, dep) {
       if (dep->pred == child) {
          gpir_node_replace_pred(dep, insert_child);
+         gpir_node_replace_child(parent, child, insert_child);
          break;
       }
    }
-   gpir_node_add_dep(insert_child, child, GPIR_DEP_INPUT);
 }
 
 void gpir_node_delete(gpir_node *node)