[Mesa-dev] [PATCH] radeon/llvm: Adapt to AMDGPU.rsq intrinsic change in LLVM 3.5

Tom Stellard tom at stellard.net
Thu Jun 19 07:05:14 PDT 2014


On Thu, Jun 19, 2014 at 03:53:42PM +0900, Michel Dänzer wrote:
> From: Michel Dänzer <michel.daenzer at amd.com>
> 

I just pushed this patch to fix the regressions.  We can update the
other intrinsics in a follow on patch.

We also need to change RSQ to use llvm.AMDGPU.rsq.clamped once
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20140616/222280.html
is committed in order to fix
https://bugs.freedesktop.org/show_bug.cgi?id=80015

Also, I forgot to do this before I committed it, but I think this patch
should go to stable.

-Tom

> Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>
> ---
>  src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c b/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
> index f8be0df..217fa32 100644
> --- a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
> +++ b/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
> @@ -1384,7 +1384,11 @@ void radeon_llvm_context_init(struct radeon_llvm_context * ctx)
>  	bld_base->op_actions[TGSI_OPCODE_UCMP].emit = emit_ucmp;
>  
>  	bld_base->rsq_action.emit = build_tgsi_intrinsic_nomem;
> +#if HAVE_LLVM >= 0x0305
> +	bld_base->rsq_action.intr_name = "llvm.AMDGPU.rsq.";
> +#else
>  	bld_base->rsq_action.intr_name = "llvm.AMDGPU.rsq";
> +#endif
>  }
>  
>  void radeon_llvm_create_func(struct radeon_llvm_context * ctx,
> -- 
> 2.0.0
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list