[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