[flang-commits] [flang] [mlir] [mlir][openmp][flang] - MLIR support for the depend clause (omp dialect) in offloading directives (PR #80626)

via flang-commits flang-commits at lists.llvm.org
Tue Feb 6 12:12:17 PST 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 34c4a0fa2b9c2181bfdbd3009e7956a50a28dab6 dfedea68f746d20a190573357b6868d0691b9617 -- flang/lib/Lower/OpenMP.cpp flang/lib/Semantics/check-omp-structure.cpp mlir/lib/Dialect/OpenMP/IR/OpenMPDialect.cpp
``````````

</details>

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

``````````diff
diff --git a/flang/lib/Semantics/check-omp-structure.cpp b/flang/lib/Semantics/check-omp-structure.cpp
index 62a8f00a2b..54101ab8a4 100644
--- a/flang/lib/Semantics/check-omp-structure.cpp
+++ b/flang/lib/Semantics/check-omp-structure.cpp
@@ -2816,12 +2816,12 @@ void OmpStructureChecker::Enter(const parser::OmpClause::Device &x) {
 void OmpStructureChecker::Enter(const parser::OmpClause::Depend &x) {
   CheckAllowed(llvm::omp::Clause::OMPC_depend);
   if ((std::holds_alternative<parser::OmpDependClause::Source>(x.v.u) ||
-      std::holds_alternative<parser::OmpDependClause::Sink>(x.v.u)) &&
+          std::holds_alternative<parser::OmpDependClause::Sink>(x.v.u)) &&
       GetContext().directive != llvm::omp::OMPD_ordered) {
     context_.Say(GetContext().clauseSource,
-                 "DEPEND(SOURCE) or DEPEND(SINK : vec) can be used only with the ordered"
-                 " directive. Used here in the %s construct."_err_en_US,
-                 parser::ToUpperCaseLetters(getDirectiveName(GetContext().directive)));
+        "DEPEND(SOURCE) or DEPEND(SINK : vec) can be used only with the ordered"
+        " directive. Used here in the %s construct."_err_en_US,
+        parser::ToUpperCaseLetters(getDirectiveName(GetContext().directive)));
   }
   if (const auto *inOut{std::get_if<parser::OmpDependClause::InOut>(&x.v.u)}) {
     const auto &designators{std::get<std::list<parser::Designator>>(inOut->t)};

``````````

</details>


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


More information about the flang-commits mailing list