From: Alyssa Rosenzweig <alyssa@rosenzweig.io>
Date: Wed, 19 Jul 2023 12:02:22 +0000 (-0400)
Subject: panfrost: Lower vertex_id for XFB
X-Git-Tag: upstream/23.3.3~5388
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=64ff2b3ed6d9c51c8087f865961e2684a170a5bc;p=platform%2Fupstream%2Fmesa.git

panfrost: Lower vertex_id for XFB

Even on Valhall, vertex_id is zero-based in a transform feedback program. Lower
that for transform feedback programs properly since it wouldn't happen
automatically on Valhall. Fixes assertion fails.

Fixes: 91ffd103510 ("pan/bi: Lower gl_VertexID in NIR")
Signed-off-by: Alyssa Rosenzweig <alyssa@rosenzweig.io>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/24198>
---

diff --git a/src/panfrost/util/pan_lower_xfb.c b/src/panfrost/util/pan_lower_xfb.c
index 3876e60..56ddd32 100644
--- a/src/panfrost/util/pan_lower_xfb.c
+++ b/src/panfrost/util/pan_lower_xfb.c
@@ -65,6 +65,20 @@ lower_xfb(nir_builder *b, nir_instr *instr, UNUSED void *data)
       return false;
 
    nir_intrinsic_instr *intr = nir_instr_as_intrinsic(instr);
+
+   /* In transform feedback programs, vertex ID becomes zero-based, so apply
+    * that lowering even on Valhall.
+    */
+   if (intr->intrinsic == nir_intrinsic_load_vertex_id) {
+      b->cursor = nir_instr_remove(instr);
+
+      nir_ssa_def *repl =
+         nir_iadd(b, nir_load_vertex_id_zero_base(b), nir_load_first_vertex(b));
+
+      nir_ssa_def_rewrite_uses(&intr->dest.ssa, repl);
+      return true;
+   }
+
    if (intr->intrinsic != nir_intrinsic_store_output)
       return false;