[Mesa-dev] [PATCH] ac/nir: remove emission of nir_op_fdiv

Timothy Arceri tarceri at itsqueeze.com
Mon Feb 5 21:23:17 UTC 2018


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

On 06/02/18 07:37, Samuel Pitoiset wrote:
> RadeonSI and RADV lower fdiv.
> 
> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
> ---
>   src/amd/common/ac_nir_to_llvm.c | 5 -----
>   1 file changed, 5 deletions(-)
> 
> diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
> index b211832b47..e25bae600f 100644
> --- a/src/amd/common/ac_nir_to_llvm.c
> +++ b/src/amd/common/ac_nir_to_llvm.c
> @@ -1760,11 +1760,6 @@ static void visit_alu(struct ac_nir_context *ctx, const nir_alu_instr *instr)
>   		src[1] = ac_to_float(&ctx->ac, src[1]);
>   		result = LLVMBuildFMul(ctx->ac.builder, src[0], src[1], "");
>   		break;
> -	case nir_op_fdiv:
> -		src[0] = ac_to_float(&ctx->ac, src[0]);
> -		src[1] = ac_to_float(&ctx->ac, src[1]);
> -		result = ac_build_fdiv(&ctx->ac, src[0], src[1]);
> -		break;
>   	case nir_op_frcp:
>   		src[0] = ac_to_float(&ctx->ac, src[0]);
>   		result = ac_build_fdiv(&ctx->ac, instr->dest.dest.ssa.bit_size == 32 ? ctx->ac.f32_1 : ctx->ac.f64_1,
> 


More information about the mesa-dev mailing list