[clang] [clang] constexpr atomic builtins (__c11_atomic_OP and __atomic_OP) (PR #98756)

[via cfe-commits]

Hana =?utf-8?q?Dusíková?= <hanicka at hanicka.net>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/98756 at github.com>


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 3b7a7f4cc43f90e79292700959c55a62ab87fd9a 000b6a5dd1b3b81a9521977acde8a6ee4d0235b0 --extensions cpp -- clang/test/SemaCXX/atomic-constexpr-c11-builtins.cpp clang/test/SemaCXX/atomic-constexpr-gcc-builtins.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 092d2dc552..5df58e17dd 100644
--- a/clang/lib/AST/ExprConstant.cpp
+++ b/clang/lib/AST/ExprConstant.cpp
@@ -15702,16 +15702,15 @@ public:
 
     case Builtin::BI__builtin_operator_delete:
       return HandleOperatorDeleteCall(Info, E);
-    
+
     case Builtin::BI__c11_atomic_thread_fence:
     case Builtin::BI__c11_atomic_signal_fence:
     case Builtin::BI__atomic_thread_fence:
-    case Builtin::BI__atomic_signal_fence:
-    {
+    case Builtin::BI__atomic_signal_fence: {
       [[maybe_unused]] APSInt IgnoredAtomicOrdering;
       return EvaluateInteger(E->getArg(0), IgnoredAtomicOrdering, Info);
     }
-   
+
     default:
       return false;
     }
@@ -17237,7 +17236,8 @@ static bool EvaluateAtomicOrder(const AtomicExpr *E, EvalInfo &Info) {
 
   if (E->getOp() != AtomicExpr::AO__c11_atomic_init) {
     const Expr *OrderSuccess = E->getOrder();
-    if (OrderSuccess && !EvaluateInteger(OrderSuccess, OrderIgnoredResult, Info))
+    if (OrderSuccess &&
+        !EvaluateInteger(OrderSuccess, OrderIgnoredResult, Info))
       return false;
   }
 

``````````

</details>


https://github.com/llvm/llvm-project/pull/98756