[flang] [llvm] [OpenMP] [TASK] Add semantic checks for detach clause (PR #119172)

via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 8 22:41:24 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 eaa1b05c48c88ed6da6b8fc21d07f8417575a051 3da984c20fc1f077f92b1542e7c037b4e16be030 --extensions cpp,h -- flang/lib/Semantics/check-omp-structure.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-structure.cpp b/flang/lib/Semantics/check-omp-structure.cpp
index 23f7624182..0c5689d8b5 100644
--- a/flang/lib/Semantics/check-omp-structure.cpp
+++ b/flang/lib/Semantics/check-omp-structure.cpp
@@ -2760,7 +2760,8 @@ void OmpStructureChecker::Leave(const parser::OmpClauseList &) {
                                                         &objs,
                                                     std::string clause) {
             for (const auto &obj : objs.v) {
-              if (const parser::Name *name{parser::Unwrap<parser::Name>(obj)}) {
+              if (const parser::Name *
+                  name{parser::Unwrap<parser::Name>(obj)}) {
                 if (name->ToString() == eventHandleSymName) {
                   context_.Say(GetContext().clauseSource,
                       "A variable: `%s` that appears in a DETACH clause cannot appear on %s clause on the same construct"_err_en_US,

``````````

</details>


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


More information about the llvm-commits mailing list