[flang-commits] [flang] [flang][OpenMP] Reassociate ATOMIC update expressions (PR #153488)

via flang-commits flang-commits at lists.llvm.org
Wed Aug 13 13:25:23 PDT 2025


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 HEAD~1 HEAD --extensions cpp,h -- flang/lib/Semantics/check-omp-atomic.cpp flang/lib/Semantics/check-omp-structure.h
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/flang/lib/Semantics/check-omp-atomic.cpp b/flang/lib/Semantics/check-omp-atomic.cpp
index 9d92be632..62bb2fdc6 100644
--- a/flang/lib/Semantics/check-omp-atomic.cpp
+++ b/flang/lib/Semantics/check-omp-atomic.cpp
@@ -115,7 +115,7 @@ struct ReassocRewriter : public evaluate::rewrite::Identity {
     auto outer1{reassocOp<T>(inner, sub[2])}; // inner + something
     auto outer2{reassocOp<T>(sub[2], inner)}; // something + inner
 #if !defined(__clang__) && !defined(_MSC_VER) && \
-      (__GNUC__ < 8 || (__GNUC__ == 8 && __GNUC_MINOR__ < 5))
+    (__GNUC__ < 8 || (__GNUC__ == 8 && __GNUC_MINOR__ < 5))
     // If GCC version < 8.5, use this definition. For the other definition
     // (which is equivalent), GCC 7.5 emits a somewhat cryptic error:
     //    use of ‘outer1’ before deduction of ‘auto’

``````````

</details>


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


More information about the flang-commits mailing list