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

Matt Arsenault arsenm2 at gmail.com
Thu Jun 19 02:01:29 PDT 2014


On Jun 18, 2014, at 11:53 PM, Michel Dänzer <michel at daenzer.net> wrote:

> From: Michel Dänzer <michel.daenzer at amd.com>
> 
> 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

While you’re at it, could you fix the old AMDIL intrinsics too?

I don’t think these have been updated yet (although the old names still technically work)

llvm.AMDIL.abs -> llvm.AMDGPU.abs
llvm.AMDIL.clamp -> llvm.AMDGPU.clamp
llvm.AMDIL.fraction -> llvm.AMDGPU.fract
llvm.AMDIL.exp -> llvm.exp2
llvm.AMDIL.round.nearest -> llvm.rint


More information about the mesa-dev mailing list