llvmpipe: fix front/back face system value in the fs
authorRoland Scheidegger <sroland@vmware.com>
Sat, 27 Aug 2022 03:26:34 +0000 (05:26 +0200)
committerRoland Scheidegger <sroland@vmware.com>
Sat, 3 Sep 2022 01:36:25 +0000 (03:36 +0200)
draw needs to inject the face information for tris in line/point fill
mode since llvmpipe only sees lines and points. But for the new style
face information (which is a system value and not an ordinary input
register) we forgot to tell draw.

Reviewed-by: Brian Paul <brianp@vmware.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/18285>

src/gallium/drivers/llvmpipe/lp_state_derived.c

index 4df9fd6..eb26821 100644 (file)
@@ -124,6 +124,18 @@ compute_vertex_info(struct llvmpipe_context *llvmpipe)
       }
    }
 
+   /*
+    * The new style front face is a system value, hence won't show up as
+    * ordinary fs register above. But we still need to assign a vs output
+    * location so draw can inject face info for unfilled tris.
+    */
+   if (llvmpipe->face_slot < 0 && fsInfo->uses_frontface) {
+      vs_index = draw_find_shader_output(llvmpipe->draw,
+                                         TGSI_SEMANTIC_FACE, 0);
+      llvmpipe->face_slot = (int)vinfo->num_attribs;
+      draw_emit_vertex_attr(vinfo, EMIT_4F, vs_index);
+   }
+
    /* Figure out if we need bcolor as well.
     */
    for (i = 0; i < 2; i++) {