[Mesa-dev] [PATCH] ac/llvm: bump the number of results to 8.

Timothy Arceri tarceri at itsqueeze.com
Tue Jan 30 06:34:32 UTC 2018


Thanks!

Reviewed-by: Timothy Arceri <tarceri at itsqueeze.com>

On 30/01/18 14:59, Dave Airlie wrote:
> From: Dave Airlie <airlied at redhat.com>
> 
> This function can get access for a 64-bit dvec4, which means we
> have to load 8 components.
> 
> This fixes:
> R600_DEBUG=nir ./bin/shader_runner generated_tests/spec/arb_gpu_shader_fp64/execution/built-in-functions/fs-abs-dvec4.shader_test -auto
> ---
>   src/amd/common/ac_llvm_build.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c
> index 5e08508..6afe7f9 100644
> --- a/src/amd/common/ac_llvm_build.c
> +++ b/src/amd/common/ac_llvm_build.c
> @@ -983,7 +983,7 @@ ac_build_buffer_load(struct ac_llvm_context *ctx,
>   	if (allow_smem && !glc && !slc) {
>   		assert(vindex == NULL);
>   
> -		LLVMValueRef result[4];
> +		LLVMValueRef result[8];
>   
>   		for (int i = 0; i < num_channels; i++) {
>   			if (i) {
> 


More information about the mesa-dev mailing list