[Mesa-dev] [PATCH] ac: replace llvm.AMDGPU.kilp by llvm.amdgcn.kill with LLVM 6

Samuel Pitoiset samuel.pitoiset at gmail.com
Thu Jan 11 15:45:11 UTC 2018


This also replaces llvm.AMDGPU.kilp by llvm.AMDGPU.kill with
LLVM < 6. Similar to RadeonSI codepath.

Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
---
 src/amd/common/ac_nir_to_llvm.c | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index d1247a7a60..0f72e132be 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/common/ac_nir_to_llvm.c
@@ -3836,14 +3836,20 @@ static void emit_barrier(struct ac_llvm_context *ac, gl_shader_stage stage)
 			   ac->voidt, NULL, 0, AC_FUNC_ATTR_CONVERGENT);
 }
 
-static void emit_discard_if(struct ac_nir_context *ctx,
-			    const nir_intrinsic_instr *instr)
+static void emit_discard(struct ac_nir_context *ctx,
+			 const nir_intrinsic_instr *instr)
 {
 	LLVMValueRef cond;
 
-	cond = LLVMBuildICmp(ctx->ac.builder, LLVMIntEQ,
-			     get_src(ctx, instr->src[0]),
-			     ctx->ac.i32_0, "");
+	if (instr->intrinsic == nir_intrinsic_discard_if) {
+		cond = LLVMBuildICmp(ctx->ac.builder, LLVMIntEQ,
+				     get_src(ctx, instr->src[0]),
+				     ctx->ac.i32_0, "");
+	} else {
+		assert(instr->intrinsic == nir_intrinsic_discard);
+		cond = LLVMConstInt(ctx->ac.i1, false, 0);
+	}
+
 	ac_build_kill_if_false(&ctx->ac, cond);
 }
 
@@ -4317,12 +4323,8 @@ static void visit_intrinsic(struct ac_nir_context *ctx,
 		result = visit_image_size(ctx, instr);
 		break;
 	case nir_intrinsic_discard:
-		ac_build_intrinsic(&ctx->ac, "llvm.AMDGPU.kilp",
-				   LLVMVoidTypeInContext(ctx->ac.context),
-				   NULL, 0, AC_FUNC_ATTR_LEGACY);
-		break;
 	case nir_intrinsic_discard_if:
-		emit_discard_if(ctx, instr);
+		emit_discard(ctx, instr);
 		break;
 	case nir_intrinsic_memory_barrier:
 	case nir_intrinsic_group_memory_barrier:
-- 
2.15.1



More information about the mesa-dev mailing list