diff options
author | Dave Airlie <[email protected]> | 2017-10-26 15:28:41 +1000 |
---|---|---|
committer | Dave Airlie <[email protected]> | 2017-10-26 15:59:31 +1000 |
commit | 82d47b9d38f22bc26001253de7996b9b372e9d0c (patch) | |
tree | b9915df61826d3ae184b6ee04244d0be3da68554 /src/amd/common/ac_llvm_build.c | |
parent | de2b241111a0d89bf4298cca8f19e9a11b8fdb6c (diff) |
ac/llvm: consolidate find lsb function.
This was the same between si and ac.
Reviewed-by: Timothy Arceri <[email protected]>
Signed-off-by: Dave Airlie <[email protected]>
Diffstat (limited to 'src/amd/common/ac_llvm_build.c')
-rw-r--r-- | src/amd/common/ac_llvm_build.c | 31 |
1 files changed, 31 insertions, 0 deletions
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c index e54806a33fe..ea238fa006e 100644 --- a/src/amd/common/ac_llvm_build.c +++ b/src/amd/common/ac_llvm_build.c @@ -1774,3 +1774,34 @@ void ac_lds_store(struct ac_llvm_context *ctx, ac_build_indexed_store(ctx, ctx->lds, dw_addr, value); } + +LLVMValueRef ac_find_lsb(struct ac_llvm_context *ctx, + LLVMTypeRef dst_type, + LLVMValueRef src0) +{ + LLVMValueRef params[2] = { + src0, + + /* The value of 1 means that ffs(x=0) = undef, so LLVM won't + * add special code to check for x=0. The reason is that + * the LLVM behavior for x=0 is different from what we + * need here. However, LLVM also assumes that ffs(x) is + * in [0, 31], but GLSL expects that ffs(0) = -1, so + * a conditional assignment to handle 0 is still required. + * + * The hardware already implements the correct behavior. + */ + LLVMConstInt(ctx->i1, 1, false), + }; + + LLVMValueRef lsb = ac_build_intrinsic(ctx, "llvm.cttz.i32", ctx->i32, + params, 2, + AC_FUNC_ATTR_READNONE); + + /* TODO: We need an intrinsic to skip this conditional. */ + /* Check for zero: */ + return LLVMBuildSelect(ctx->builder, LLVMBuildICmp(ctx->builder, + LLVMIntEQ, src0, + ctx->i32_0, ""), + LLVMConstInt(ctx->i32, -1, 0), lsb, ""); +} |