commit b43b1535e25b3e7ec59065235ccc466c7de42dd3
author Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com> Wed Oct 14 20:45:19 2020 -0400
committer Marge Bot <eric+marge@anholt.net> Fri Oct 23 14:48:23 2020 +0000
tree fa74c613f7fec02117ba4c0b63fd8053d4e7c86d
parent e33b2976f3adcb8c7185265efe2812d91b33ab64

pan/bi: Implement spilling

Now that all the helpers are in place, we can wire it up.

Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7206>

src/panfrost/bifrost/bi_ra.c

diff --git a/src/panfrost/bifrost/bi_ra.c b/src/panfrost/bifrost/bi_ra.c
index 82ecb3c..b586969 100644
--- a/src/panfrost/bifrost/bi_ra.c
+++ b/src/panfrost/bifrost/bi_ra.c
@@ -349,6 +349,9 @@
 
         unsigned iter_count = 100; /* max iterations */
 
+        /* Number of bytes of memory we've spilled into */
+        unsigned spill_count = 0;
+
         /* For instructions that both read and write from a data register, it's
          * the *same* data register. We enforce that constraint by just doing a
          * quick rewrite. TODO: are there cases where this causes RA to have no
@@ -372,13 +375,15 @@
                         if (spill_node == -1)
                                 unreachable("Failed to choose spill node\n");
 
-                        unreachable("Spilling not implemented");
+                        spill_count += bi_spill_register(ctx, spill_node, spill_count);
                 }
 
                 bi_invalidate_liveness(ctx);
                 l = bi_allocate_registers(ctx, &success);
         } while(!success && ((iter_count--) > 0));
 
+        assert(success);
+
         bi_install_registers(ctx, l);
 
         lcra_free(l);