[Mesa-dev,v2,10/23] i965/gs: Set number of control data bits for stream mode.

Submitted by Iago Toral Quiroga on June 18, 2014, 9:51 a.m.

Details

Message ID 1403085110-31168-11-git-send-email-itoral@igalia.com
State Superseded
Headers show

Not browsing as part of any series.

Commit Message

Iago Toral Quiroga June 18, 2014, 9:51 a.m.
If the geometry shader is indeed using streams then we need 2 control data
bits per vertex for the StreamID. If the shader is not using streams then
we don't need control data bits.
---
 src/mesa/drivers/dri/i965/brw_vec4_gs.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

Patch hide | download patch | download mbox

diff --git a/src/mesa/drivers/dri/i965/brw_vec4_gs.c b/src/mesa/drivers/dri/i965/brw_vec4_gs.c
index 4b1ba36..6428291 100644
--- a/src/mesa/drivers/dri/i965/brw_vec4_gs.c
+++ b/src/mesa/drivers/dri/i965/brw_vec4_gs.c
@@ -81,10 +81,11 @@  do_gs_prog(struct brw_context *brw,
        */
       c.prog_data.control_data_format = GEN7_GS_CONTROL_DATA_FORMAT_GSCTL_SID;
 
-      /* However, StreamID is not yet supported, so we output zero bits of
-       * control data per vertex.
-       */
-      c.control_data_bits_per_vertex = 0;
+      /* We only have to emit control bits if we are using streams */
+      if (prog->Geom.UsesStreams)
+         c.control_data_bits_per_vertex = 2;
+      else
+         c.control_data_bits_per_vertex = 0;
    } else {
       /* When the output type is triangle_strip or line_strip, EndPrimitive()
        * may be used to terminate the current strip and start a new one