r600: move per-type settings into a switch statement
authorDave Airlie <airlied@redhat.com>
Mon, 30 Nov 2015 00:37:21 +0000 (10:37 +1000)
committerDave Airlie <airlied@redhat.com>
Mon, 30 Nov 2015 01:08:00 +0000 (11:08 +1000)
This will allow adding tess stuff much cleaner later.

Reviewed-by: Glenn Kennard <glenn.kennard@gmail.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
src/gallium/drivers/r600/r600_shader.c

index 560197c82b5497101905d0b85e2c46c6907de0ba..d25fc3b85a2eb4eecda99dd4484bd9d9f4313e37 100644 (file)
@@ -1909,13 +1909,23 @@ static int r600_shader_from_tgsi(struct r600_context *rctx,
        shader->processor_type = ctx.type;
        ctx.bc->type = shader->processor_type;
 
-       if (ctx.type == TGSI_PROCESSOR_VERTEX) {
+       switch (ctx.type) {
+       case TGSI_PROCESSOR_VERTEX:
                shader->vs_as_gs_a = key.vs.as_gs_a;
                shader->vs_as_es = key.vs.as_es;
+               if (shader->vs_as_es)
+                       ring_outputs = true;
+               break;
+       case TGSI_PROCESSOR_GEOMETRY:
+               ring_outputs = true;
+               break;
+       case TGSI_PROCESSOR_FRAGMENT:
+               shader->two_side = key.ps.color_two_side;
+               break;
+       default:
+               break;
        }
 
-       ring_outputs = shader->vs_as_es || ctx.type == TGSI_PROCESSOR_GEOMETRY;
-
        if (shader->vs_as_es) {
                ctx.gs_for_vs = &rctx->gs_shader->current->shader;
        } else {
@@ -1936,8 +1946,6 @@ static int r600_shader_from_tgsi(struct r600_context *rctx,
        shader->nr_ps_color_exports = 0;
        shader->nr_ps_max_color_exports = 0;
 
-       if (ctx.type == TGSI_PROCESSOR_FRAGMENT)
-               shader->two_side = key.ps.color_two_side;
 
        /* register allocations */
        /* Values [0,127] correspond to GPR[0..127].