[clang] [OpenACC] Implement `self` clause for compute constructs (PR #88760)

Alexey Bataev via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 16 04:34:37 PDT 2024


================
@@ -835,18 +835,23 @@ Parser::OpenACCClauseParseResult Parser::ParseOpenACCClauseParams(
     case OpenACCClauseKind::Default: {
       Token DefKindTok = getCurToken();
 
-      if (expectIdentifierOrKeyword(*this))
-        break;
+      if (expectIdentifierOrKeyword(*this)) {
+        Parens.skipToEnd();
+        return OpenACCCanContinue();
+      }
 
       ConsumeToken();
 
       OpenACCDefaultClauseKind DefKind =
           getOpenACCDefaultClauseKind(DefKindTok);
 
-      if (DefKind == OpenACCDefaultClauseKind::Invalid)
+      if (DefKind == OpenACCDefaultClauseKind::Invalid) {
         Diag(DefKindTok, diag::err_acc_invalid_default_clause_kind);
-      else
+        Parens.skipToEnd();
+        return OpenACCCanContinue();
+      } else {
         ParsedClause.setDefaultDetails(DefKind);
+      }
----------------
alexey-bataev wrote:

No need for else here

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


More information about the cfe-commits mailing list