Mesa (master): draw: use correct output prim for non-adjacent topologies in prim assembler.

[Roland Scheidegger]

Module: Mesa
Branch: master
Commit: fe86415beb8edfe3a6a4300ca537fae1bbad64a7
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=fe86415beb8edfe3a6a4300ca537fae1bbad64a7

Author: Roland Scheidegger <sroland at vmware.com>
Date:   Mon Dec  1 23:40:31 2014 +0100

draw: use correct output prim for non-adjacent topologies in prim assembler.

The decomposition done in the prim assembler will turn tri fans into tris,
but this wasn't reflected in the output prim type. Meaning with a tri fan
with 6 verts input, the output was a tri fan with 12 vertices instead of a
tri list with 12 vertices (not as bad as it sounds, since the additional tris
created would all be degenerate since they'd all have two times vertex zero
but still bogus).
This is because the prim assembler is used if either the input topology is
something with adjacency, or if prim id needs to be injected, and for the
latter case topologies without adjacency can be converted to basic ones.
Unfortunately decomposition here for inserting prim ids is necessary, at
least for the indexed case where we can't just insert the prim id at the
right place depending on provoking vertex.

Reviewed-by: Jose Fonseca <jfonseca at vmware.com>

---

 src/gallium/auxiliary/draw/draw_prim_assembler.c |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/gallium/auxiliary/draw/draw_prim_assembler.c b/src/gallium/auxiliary/draw/draw_prim_assembler.c
index c609a77..5cb19f5 100644
--- a/src/gallium/auxiliary/draw/draw_prim_assembler.c
+++ b/src/gallium/auxiliary/draw/draw_prim_assembler.c
@@ -113,8 +113,8 @@ copy_verts(struct draw_assembler *asmblr,
 
 static void
 inject_primid(struct draw_assembler *asmblr,
-                      unsigned idx,
-                      unsigned primid)
+              unsigned idx,
+              unsigned primid)
 {
    int slot = asmblr->primid_slot;
    char *input = (char*)asmblr->input_verts->verts;
@@ -246,7 +246,7 @@ draw_prim_assembler_run(struct draw_context *draw,
 {
    struct draw_assembler *asmblr = draw->ia;
    unsigned start, i;
-   unsigned assembled_prim = u_assembled_prim(input_prims->prim);
+   unsigned assembled_prim = u_reduced_prim(input_prims->prim);
    unsigned max_primitives = u_decomposed_prims_for_vertices(
       input_prims->prim, input_prims->count);
    unsigned max_verts = u_vertices_per_prim(assembled_prim) * max_primitives;
@@ -268,7 +268,7 @@ draw_prim_assembler_run(struct draw_context *draw,
    output_prims->linear = TRUE;
    output_prims->elts = NULL;
    output_prims->start = 0;
-   output_prims->prim = u_assembled_prim(input_prims->prim);
+   output_prims->prim = assembled_prim;
    output_prims->flags = 0x0;
    output_prims->primitive_lengths = MALLOC(sizeof(unsigned));
    output_prims->primitive_lengths[0] = 0;