[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