Mesa (master): ac/nir: remove unused code for nir_op_{fmod,frem}

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Thu Oct 3 16:12:08 UTC 2019


Module: Mesa
Branch: master
Commit: d861401554b52b2c2fc6721c69bdfe1697ee608f
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=d861401554b52b2c2fc6721c69bdfe1697ee608f

Author: Samuel Pitoiset <samuel.pitoiset at gmail.com>
Date:   Thu Oct  3 16:21:47 2019 +0200

ac/nir: remove unused code for nir_op_{fmod,frem}

RADV and RadeonSI both lower these two NIR instructions.

Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>

---

 src/amd/common/ac_nir_to_llvm.c | 14 --------------
 1 file changed, 14 deletions(-)

diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index ea7bf91b79b..bb99c736f78 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/common/ac_nir_to_llvm.c
@@ -548,20 +548,6 @@ static void visit_alu(struct ac_nir_context *ctx, const nir_alu_instr *instr)
 	case nir_op_umod:
 		result = LLVMBuildURem(ctx->ac.builder, src[0], src[1], "");
 		break;
-	case nir_op_fmod:
-		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]);
-		result = emit_intrin_1f_param(&ctx->ac, "llvm.floor",
-		                              ac_to_float_type(&ctx->ac, def_type), result);
-		result = LLVMBuildFMul(ctx->ac.builder, src[1] , result, "");
-		result = LLVMBuildFSub(ctx->ac.builder, src[0], result, "");
-		break;
-	case nir_op_frem:
-		src[0] = ac_to_float(&ctx->ac, src[0]);
-		src[1] = ac_to_float(&ctx->ac, src[1]);
-		result = LLVMBuildFRem(ctx->ac.builder, src[0], src[1], "");
-		break;
 	case nir_op_irem:
 		result = LLVMBuildSRem(ctx->ac.builder, src[0], src[1], "");
 		break;




More information about the mesa-commit mailing list