[Mesa-dev] [PATCH 2/2] radeonsi/nir: fix tess varying loads for doubles

[Marek Olšák]

For the series:

Reviewed-by: Marek Olšák <marek.olsak at amd.com>

Marek

On Wed, Feb 21, 2018 at 12:42 AM, Timothy Arceri <tarceri at itsqueeze.com> wrote:
> Fixes the following piglit tests:
>
> tests/spec/arb_tessellation_shader/execution/double-array-vs-tcs-tes.shader_test
> tests/spec/arb_tessellation_shader/execution/double-vs-tcs-tes.shader_test
> ---
>  src/gallium/drivers/radeonsi/si_shader.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/gallium/drivers/radeonsi/si_shader.c b/src/gallium/drivers/radeonsi/si_shader.c
> index 33319b249c..9ccae9f18d 100644
> --- a/src/gallium/drivers/radeonsi/si_shader.c
> +++ b/src/gallium/drivers/radeonsi/si_shader.c
> @@ -1273,7 +1273,7 @@ static LLVMValueRef si_nir_load_tcs_varyings(struct ac_shader_abi *abi,
>
>         LLVMValueRef value[4];
>         for (unsigned i = 0; i < num_components + component; i++) {
> -               value[i] = lds_load(bld_base, ctx->i32, i, dw_addr);
> +               value[i] = lds_load(bld_base, type, i, dw_addr);
>         }
>
>         return ac_build_varying_gather_values(&ctx->ac, value, num_components, component);
> @@ -1360,7 +1360,7 @@ LLVMValueRef si_nir_load_input_tes(struct ac_shader_abi *abi,
>          */
>         LLVMValueRef value[4];
>         for (unsigned i = component; i < num_components + component; i++) {
> -               value[i] = buffer_load(&ctx->bld_base, ctx->i32, i, buffer, base, addr, true);
> +               value[i] = buffer_load(&ctx->bld_base, type, i, buffer, base, addr, true);
>         }
>
>         return ac_build_varying_gather_values(&ctx->ac, value, num_components, component);
> --
> 2.14.3
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev