diff options
| author | Marek Olšák <[email protected]> | 2018-08-24 23:00:37 -0400 |
|---|---|---|
| committer | Marek Olšák <[email protected]> | 2018-09-10 15:19:56 -0400 |
| commit | a5f35aa742c3f1e2fae6a6c2fb53f92822f0cb70 (patch) | |
| tree | 3ab935b7abf30d81d151ab2b258805d87b366429 /src/amd | |
| parent | 662db03577377cd0acca279ca482ea8e9a4e1704 (diff) | |
ac: revert new LLVM 7.0 behavior for fdiv
Cc: 18.2 <[email protected]>
Tested-by: Dieter Nützel <[email protected]>
Diffstat (limited to 'src/amd')
| -rw-r--r-- | src/amd/common/ac_llvm_build.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c index c741a1ab62d..629cd2a7527 100644 --- a/src/amd/common/ac_llvm_build.c +++ b/src/amd/common/ac_llvm_build.c @@ -561,7 +561,14 @@ ac_build_fdiv(struct ac_llvm_context *ctx, LLVMValueRef num, LLVMValueRef den) { - LLVMValueRef ret = LLVMBuildFDiv(ctx->builder, num, den, ""); + /* If we do (num / den), LLVM >= 7.0 does: + * return num * v_rcp_f32(den * (fabs(den) > 0x1.0p+96f ? 0x1.0p-32f : 1.0f)); + * + * If we do (num * (1 / den)), LLVM does: + * return num * v_rcp_f32(den); + */ + LLVMValueRef rcp = LLVMBuildFDiv(ctx->builder, ctx->f32_1, den, ""); + LLVMValueRef ret = LLVMBuildFMul(ctx->builder, num, rcp, ""); /* Use v_rcp_f32 instead of precise division. */ if (!LLVMIsConstant(ret)) |