[Mesa-dev] [PATCH] ac: fix return type for llvm.amdgcn.frexp.exp.i32.64

Timothy Arceri tarceri at itsqueeze.com
Fri Mar 29 07:43:18 UTC 2019


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

On 29/3/19 6:39 pm, Samuel Pitoiset wrote:
> This fixes the following piglit with RadeonSI
> tests/spec/arb_gpu_shader_fp64/execution/built-in-functions/fs-frexp-dvec4.shader_test
> 
> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
> ---
>   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 5572b244720..eb9e4504248 100644
> --- a/src/amd/common/ac_llvm_build.c
> +++ b/src/amd/common/ac_llvm_build.c
> @@ -3942,7 +3942,7 @@ ac_build_frexp_exp(struct ac_llvm_context *ctx, LLVMValueRef src0,
>   		type = ctx->i32;
>   	} else {
>   		intr = "llvm.amdgcn.frexp.exp.i32.f64";
> -		type = ctx->i64;
> +		type = ctx->i32;
>   	}
>   
>   	LLVMValueRef params[] = {
> 


More information about the mesa-dev mailing list