[Mesa-dev] [PATCH v2] ac: fix f2b and i2b for doubles

[Timothy Arceri]

Without this llvm was asserting in debug builds.

V2: use LLVMConstNull()
---
 src/amd/common/ac_nir_to_llvm.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index f37f19404e..789928761b 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/common/ac_nir_to_llvm.c
@@ -1437,8 +1437,9 @@ static LLVMValueRef emit_f2b(struct ac_llvm_context *ctx,
                             LLVMValueRef src0)
 {
        src0 = ac_to_float(ctx, src0);
+       LLVMValueRef zero = LLVMConstNull(LLVMTypeOf(src0));
        return LLVMBuildSExt(ctx->builder,
-                            LLVMBuildFCmp(ctx->builder, LLVMRealUNE, src0, 
ctx->f32_0, ""),
+                            LLVMBuildFCmp(ctx->builder, LLVMRealUNE, src0, 
zero, ""),
                             ctx->i32, "");
 }
 
@@ -1457,8 +1458,9 @@ static LLVMValueRef emit_b2i(struct ac_llvm_context *ctx,
 static LLVMValueRef emit_i2b(struct ac_llvm_context *ctx,
                             LLVMValueRef src0)
 {
+       LLVMValueRef zero = LLVMConstNull(LLVMTypeOf(src0));
        return LLVMBuildSExt(ctx->builder,
-                            LLVMBuildICmp(ctx->builder, LLVMIntNE, src0, 
ctx->i32_0, ""),
+                            LLVMBuildICmp(ctx->builder, LLVMIntNE, src0, zero, 
""),
                             ctx->i32, "");
 }
 
-- 
2.14.3

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev