freedreno/a6xx: fix binning pass vs. xfb

We could bit doing streamout from binning pass.  In this case we want to
use the full VS which doesn't have (potentially streamed out) varyings
stripped out.

Signed-off-by: Rob Clark <robdclark@chromium.org>
Reviewed-by: Eric Anholt <eric@anholt.net>
Reviewed-by: Kristian H. Kristensen <hoegsberg@google.com>

diff --git a/src/gallium/drivers/freedreno/a6xx/fd6_program.c b/src/gallium/drivers/freedreno/a6xx/fd6_program.c
index 08c264b4f116f8270f98f767bcf1f3ee67b961be..6f29775d9aab868b00023a25eec190a9c4dfa8bc 100644 (file)
--- a/src/gallium/drivers/freedreno/a6xx/fd6_program.c
+++ b/src/gallium/drivers/freedreno/a6xx/fd6_program.c
@@ -321,7 +321,7 @@ setup_stateobj(struct fd_ringbuffer *ring, struct fd_screen *screen,
        struct ir3_shader_linkage l = {0};
        ir3_link_shaders(&l, vs, fs);
 
-       if ((vs->shader->stream_output.num_outputs > 0) && !binning_pass)
+       if (vs->shader->stream_output.num_outputs > 0)
                link_stream_out(&l, vs);
 
        BITSET_DECLARE(varbs, 128) = {0};
@@ -348,7 +348,7 @@ setup_stateobj(struct fd_ringbuffer *ring, struct fd_screen *screen,
                ir3_link_add(&l, psize_regid, 0x1, l.max_loc);
        }
 
-       if ((vs->shader->stream_output.num_outputs > 0) && !binning_pass) {
+       if (vs->shader->stream_output.num_outputs > 0) {
                setup_stream_out(state, vs, &l);
        }
 
@@ -656,7 +656,11 @@ fd6_program_create(void *data, struct ir3_shader_variant *bs,
        struct fd_context *ctx = data;
        struct fd6_program_state *state = CALLOC_STRUCT(fd6_program_state);
 
-       state->bs = bs;
+       /* if we have streamout, use full VS in binning pass, as the
+        * binning pass VS will have outputs on other than position/psize
+        * stripped out:
+        */
+       state->bs = vs->shader->stream_output.num_outputs ? vs : bs;
        state->vs = vs;
        state->fs = fs;
        state->config_stateobj = fd_ringbuffer_new_object(ctx->pipe, 0x1000);