commit ad8a8499c9b95072d4bb95bed05bad67cf0a7614
author Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com> Wed Oct 14 18:53:59 2020 -0400
committer Marge Bot <eric+marge@anholt.net> Fri Oct 23 14:48:23 2020 +0000
tree f2a118fb426c95bc89493d09b2d7bbc7fb1c9c0d
parent 8477678cfeb3ab0d80983ec5901742baf786d8cf

pan/bi: Stub spilling

Like Midgard.

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 7a3b307..33be88a 100644
--- a/src/panfrost/bifrost/bi_ra.c
+++ b/src/panfrost/bifrost/bi_ra.c
@@ -180,6 +180,8 @@
         struct lcra_state *l = NULL;
         bool success = false;
 
+        unsigned iter_count = 100; /* max iterations */
+
         /* 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
@@ -197,14 +199,12 @@
                 if (l) {
                         lcra_free(l);
                         l = NULL;
+                        unreachable("Spilling not implemented");
                 }
 
                 bi_invalidate_liveness(ctx);
                 l = bi_allocate_registers(ctx, &success);
-
-                /* TODO: Spilling */
-                assert(success);
-        } while(!success);
+        } while(!success && ((iter_count--) > 0));
 
         bi_install_registers(ctx, l);