From 24575c815c3a8bf3457b54077b93baa5fd73c7f2 Mon Sep 17 00:00:00 2001 From: Timothy Arceri Date: Thu, 11 Jan 2018 17:04:22 +1100 Subject: [PATCH] ac/nir: fix translation of nir_op_frsq for doubles Without this we end up with the llvm error message: "Both operands to a binary operator are not of the same type!" Reviewed-by: Samuel Pitoiset --- src/amd/common/ac_nir_to_llvm.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c index 7153c9708d4..372a424ce51 100644 --- a/src/amd/common/ac_nir_to_llvm.c +++ b/src/amd/common/ac_nir_to_llvm.c @@ -1842,7 +1842,8 @@ static void visit_alu(struct ac_nir_context *ctx, const nir_alu_instr *instr) case nir_op_frsq: result = emit_intrin_1f_param(&ctx->ac, "llvm.sqrt", ac_to_float_type(&ctx->ac, def_type), src[0]); - result = ac_build_fdiv(&ctx->ac, ctx->ac.f32_1, result); + result = ac_build_fdiv(&ctx->ac, instr->dest.dest.ssa.bit_size == 32 ? ctx->ac.f32_1 : ctx->ac.f64_1, + result); break; case nir_op_fpow: result = emit_intrin_2f_param(&ctx->ac, "llvm.pow", -- 2.30.2