[clang] [C++23] [CLANG] Adding C++23 constexpr math functions: fmin and frexp. (PR #88978)
via cfe-commits
cfe-commits at lists.llvm.org
Wed May 1 08:50:08 PDT 2024
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff a3efc53f168b1451803a40075201c3490d6e3928 625028aed5ad85922f082f5192abd5ceedb0b68d -- clang/test/CodeGenCXX/constexpr-math.cpp clang/test/SemaCXX/constexpr-math.cpp clang/lib/AST/ExprConstant.cpp
``````````
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/clang/lib/AST/ExprConstant.cpp b/clang/lib/AST/ExprConstant.cpp
index 4e66490596..fef6b2fd1d 100644
--- a/clang/lib/AST/ExprConstant.cpp
+++ b/clang/lib/AST/ExprConstant.cpp
@@ -14558,7 +14558,7 @@ bool FloatExprEvaluator::VisitCallExpr(const CallExpr *E) {
int FrexpExp;
FrexpExp = ilogb(Result);
FrexpExp = FrexpExp == llvm::detail::IEEEFloat::IEK_Zero ? 0 : FrexpExp + 1;
- Result = scalbn(Result, -FrexpExp, RM);
+ Result = scalbn(Result, -FrexpExp, RM);
return true;
}
case Builtin::BI__builtin_huge_val:
``````````
</details>
https://github.com/llvm/llvm-project/pull/88978
More information about the cfe-commits
mailing list