radv: fix adjusting vertex fetches since 16bit support
authorSamuel Pitoiset <samuel.pitoiset@gmail.com>
Wed, 25 Jul 2018 12:55:31 +0000 (14:55 +0200)
committerSamuel Pitoiset <samuel.pitoiset@gmail.com>
Thu, 26 Jul 2018 06:57:43 +0000 (08:57 +0200)
Move the integer conversion after the fixup.

This fixes some regressions with
dEQP-VK.pipeline.vertex_input.single_attribute.mat4.as_a2r10g10b10*

Fixes: b722b29f10 ("radv: add support for 16bit input/output")
Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
src/amd/vulkan/radv_nir_to_llvm.c

index 9f9dc0d4fee11eaa95a9cb9eab767bef351414a0..64b6522cd93a672db7f05f555c562622677a4756 100644 (file)
@@ -2030,15 +2030,16 @@ handle_vs_input_decl(struct radv_shader_context *ctx,
                                output[chan] = LLVMBuildBitCast(ctx->ac.builder, output[chan], ctx->ac.f32, "");
                                output[chan] = LLVMBuildFPTrunc(ctx->ac.builder, output[chan], ctx->ac.f16, "");
                        }
-                       output[chan] = ac_to_integer(&ctx->ac, output[chan]);
-                       if (type == GLSL_TYPE_UINT16 || type == GLSL_TYPE_INT16)
-                               output[chan] = LLVMBuildTrunc(ctx->ac.builder, output[chan], ctx->ac.i16, "");
                }
 
                unsigned alpha_adjust = (ctx->options->key.vs.alpha_adjust >> (attrib_index * 2)) & 3;
                output[3] = adjust_vertex_fetch_alpha(ctx, alpha_adjust, output[3]);
 
                for (unsigned chan = 0; chan < 4; chan++) {
+                       output[chan] = ac_to_integer(&ctx->ac, output[chan]);
+                       if (type == GLSL_TYPE_UINT16 || type == GLSL_TYPE_INT16)
+                               output[chan] = LLVMBuildTrunc(ctx->ac.builder, output[chan], ctx->ac.i16, "");
+
                        ctx->inputs[ac_llvm_reg_index_soa(variable->data.location + i, chan)] = output[chan];
                }
        }