ac/llvm: fix type of second llvm.cttz.* parameter
authorNicolai Hähnle <nicolai.haehnle@amd.com>
Sat, 24 Jun 2017 15:56:38 +0000 (17:56 +0200)
committerDave Airlie <airlied@redhat.com>
Tue, 27 Jun 2017 00:28:30 +0000 (10:28 +1000)
LLVM has required an i1 here for a long time. llvm.ctlz.* was fixed in
commit edd23e06067 ("ac/llvm: fix various findMSB bugs").

Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
Signed-off-by: Dave Airlie <airlied@redhat.com>
src/amd/common/ac_nir_to_llvm.c

index f0ae7087379d3e99b3015e558acdb27042abff49..35f66e849d0796a42019d1df3f57c9fc2ba37ba2 100644 (file)
@@ -1190,7 +1190,7 @@ static LLVMValueRef emit_find_lsb(struct nir_to_llvm_context *ctx,
                 *
                 * The hardware already implements the correct behavior.
                 */
-               LLVMConstInt(ctx->i32, 1, false),
+               LLVMConstInt(ctx->i1, 1, false),
        };
        return ac_build_intrinsic(&ctx->ac, "llvm.cttz.i32", ctx->i32, params, 2, AC_FUNC_ATTR_READNONE);
 }