[clang] [OpenACC] Add 'clause' parsing infrastructure plus a few clauses (PR #75052)

via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 11 06:50:23 PST 2023


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 29bd78b2f61e638f6c26bc417ae476754b91e985 0bcee977f95f87a5ccf7bc53d6cd0a5d9521d963 -- clang/test/ParserOpenACC/parse-clauses.c clang/include/clang/Basic/OpenACCKinds.h clang/lib/Parse/ParseOpenACC.cpp clang/test/ParserOpenACC/parse-cache-construct.c clang/test/ParserOpenACC/parse-constructs.c clang/test/ParserOpenACC/parse-wait-construct.c clang/test/ParserOpenACC/unimplemented.c clang/test/ParserOpenACC/unimplemented.cpp
``````````

</details>

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

``````````diff
diff --git a/clang/lib/Parse/ParseOpenACC.cpp b/clang/lib/Parse/ParseOpenACC.cpp
index 2ad296e23e..50a8d85450 100644
--- a/clang/lib/Parse/ParseOpenACC.cpp
+++ b/clang/lib/Parse/ParseOpenACC.cpp
@@ -80,16 +80,16 @@ OpenACCClauseKind getOpenACCClauseKind(Token Tok) {
     return OpenACCClauseKind::Invalid;
 
   return llvm::StringSwitch<OpenACCClauseKind>(
-      Tok.getIdentifierInfo()->getName())
-    .Case("finalize", OpenACCClauseKind::Finalize)
-    .Case("if_present", OpenACCClauseKind::IfPresent)
-    .Case("seq", OpenACCClauseKind::Seq)
-    .Case("independent", OpenACCClauseKind::Independent)
-    .Case("auto", OpenACCClauseKind::Auto)
-    .Case("worker", OpenACCClauseKind::Worker)
-    .Case("vector", OpenACCClauseKind::Vector)
-    .Case("nohost", OpenACCClauseKind::NoHost)
-    .Default(OpenACCClauseKind::Invalid);
+             Tok.getIdentifierInfo()->getName())
+      .Case("finalize", OpenACCClauseKind::Finalize)
+      .Case("if_present", OpenACCClauseKind::IfPresent)
+      .Case("seq", OpenACCClauseKind::Seq)
+      .Case("independent", OpenACCClauseKind::Independent)
+      .Case("auto", OpenACCClauseKind::Auto)
+      .Case("worker", OpenACCClauseKind::Worker)
+      .Case("vector", OpenACCClauseKind::Vector)
+      .Case("nohost", OpenACCClauseKind::NoHost)
+      .Default(OpenACCClauseKind::Invalid);
 }
 
 // Since 'atomic' is effectively a compound directive, this will decode the
@@ -313,7 +313,7 @@ bool ParseOpenACCClause(Parser &P) {
 // have to do this because 'SkipUntil' considers paren balancing, which isn't
 // what we want.
 void SkipUntilEndOfDirective(Parser &P) {
-  while(P.getCurToken().isNot(tok::annot_pragma_openacc_end))
+  while (P.getCurToken().isNot(tok::annot_pragma_openacc_end))
     P.ConsumeAnyToken();
 }
 
@@ -330,7 +330,8 @@ void ParseOpenACCClauseList(Parser &P) {
       P.ConsumeToken();
     FirstClause = false;
 
-    // Recovering from a bad clause is really difficult, so we just give up on error.
+    // Recovering from a bad clause is really difficult, so we just give up on
+    // error.
     if (ParseOpenACCClause(P)) {
       SkipUntilEndOfDirective(P);
       return;

``````````

</details>


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


More information about the cfe-commits mailing list