pan/midgard: Don't use no_spill for memory spill src

I'm not totally sure why this would *break* things, but it's certainly
not necessary and it does break things. Somehow this gives the RA more
freedom, fixing some spill issues.

Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>

diff --git a/src/panfrost/midgard/midgard_ra.c b/src/panfrost/midgard/midgard_ra.c
index 9749ebb9ceba0566e1aa0d04c84e17ce3bf852f2..2fe406d6e7aec77e495942ead488594136428f36 100644 (file)
--- a/src/panfrost/midgard/midgard_ra.c
+++ b/src/panfrost/midgard/midgard_ra.c
@@ -691,8 +691,10 @@ mir_choose_spill_node(
                 if (ins->no_spill & (1 << l->spill_class)) {
                         lcra_set_node_spill_cost(l, ins->dest, -1);
 
-                        mir_foreach_src(ins, s)
-                                lcra_set_node_spill_cost(l, ins->src[s], -1);
+                        if (l->spill_class != REG_CLASS_WORK) {
+                                mir_foreach_src(ins, s)
+                                        lcra_set_node_spill_cost(l, ins->src[s], -1);
+                        }
                 }
         }