From: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Date: Tue, 3 Mar 2020 13:09:18 +0000 (-0500)
Subject: pan/bi: Add clause header fields to bi_clause
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=fba1d12742db36536b6010807a59884abfb79973;p=mesa.git

pan/bi: Add clause header fields to bi_clause

These will be filled out during scheduling (and possibly RA), to be used
when emitting code.

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

diff --git a/src/panfrost/bifrost/compiler.h b/src/panfrost/bifrost/compiler.h
index d4ac239347a..ed9dd751910 100644
--- a/src/panfrost/bifrost/compiler.h
+++ b/src/panfrost/bifrost/compiler.h
@@ -212,6 +212,25 @@ typedef struct {
                 bi_instruction *instructions[16];
                 bi_bundle bundles[8];
         };
+
+        /* For scoreboarding -- the clause ID (this is not globally unique!)
+         * and its dependencies in terms of other clauses, computed during
+         * scheduling and used when emitting code. Dependencies expressed as a
+         * bitfield matching the hardware, except shifted by a clause (the
+         * shift back to the ISA's off-by-one encoding is worked out when
+         * emitting clauses) */
+        unsigned scoreboard_id;
+        uint8_t dependencies;
+
+        /* Back-to-back corresponds directly to the back-to-back bit. Branch
+         * conditional corresponds to the branch conditional bit except that in
+         * the emitted code it's always set if back-to-bit is, whereas we use
+         * the actual value (without back-to-back so to speak) internally */
+        bool back_to_back;
+        bool branch_conditional;
+
+        /* Corresponds to the usual bit but shifted by a clause */
+        bool data_register_write_barrier;
 } bi_clause;
 
 typedef struct bi_block {