radeonsi: si_llvm_emit_vs_epilogue is never used with gs copy shaders
authorNicolai Hähnle <nicolai.haehnle@amd.com>
Fri, 22 Jan 2016 22:26:44 +0000 (17:26 -0500)
committerNicolai Hähnle <nicolai.haehnle@amd.com>
Mon, 25 Jan 2016 15:16:02 +0000 (10:16 -0500)
Hence remove the misleading branch on is_gs_copy_shader.

Reviewed-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
src/gallium/drivers/radeonsi/si_shader.c

index bdaba74..38ed70c 100644 (file)
@@ -2152,6 +2152,8 @@ static void si_llvm_emit_vs_epilogue(struct lp_build_tgsi_context * bld_base)
        struct si_shader_output_values *outputs = NULL;
        int i,j;
 
+       assert(!si_shader_ctx->is_gs_copy_shader);
+
        outputs = MALLOC((info->num_outputs + 1) * sizeof(outputs[0]));
 
        /* Vertex color clamping.
@@ -2160,8 +2162,7 @@ static void si_llvm_emit_vs_epilogue(struct lp_build_tgsi_context * bld_base)
         * an IF statement is added that clamps all colors if the constant
         * is true.
         */
-       if (si_shader_ctx->type == TGSI_PROCESSOR_VERTEX &&
-           !si_shader_ctx->is_gs_copy_shader) {
+       if (si_shader_ctx->type == TGSI_PROCESSOR_VERTEX) {
                struct lp_build_if_state if_ctx;
                LLVMValueRef cond = NULL;
                LLVMValueRef addr, val;