Revert "[OpenMP] Add the `ompx_attribute` clause for target directives"

This reverts commit ef9ec4bbcca2fa4f64df47bc426f1d1c59ea47e2.

The changes broke several bots:
https://lab.llvm.org/buildbot/#/builders/176/builds/3408
https://lab.llvm.org/buildbot/#/builders/198/builds/4028
https://lab.llvm.org/buildbot/#/builders/197/builds/8491
https://lab.llvm.org/buildbot/#/builders/197/builds/8491

1 files changed, 0 insertions, 59 deletions

diff --git a/clang/lib/Parse/ParseOpenMP.cpp b/clang/lib/Parse/ParseOpenMP.cpp
index 66cabb1..96d2e2c 100644
--- a/clang/lib/Parse/ParseOpenMP.cpp
+++ b/clang/lib/Parse/ParseOpenMP.cpp
@@ -3411,9 +3411,6 @@ OMPClause *Parser::ParseOpenMPClause(OpenMPDirectiveKind DKind,
           << getOpenMPClauseName(CKind) << getOpenMPDirectiveName(DKind);
     SkipUntil(tok::comma, tok::annot_pragma_openmp_end, StopBeforeMatch);
     break;
-  case OMPC_ompx_attribute:
-    Clause = ParseOpenMPOMPXAttributesClause(WrongDirective);
-    break;
   default:
     break;
   }
@@ -3694,62 +3691,6 @@ OMPClause *Parser::ParseOpenMPInteropClause(OpenMPClauseKind Kind,
   llvm_unreachable("Unexpected interop variable clause.");
 }
 
-OMPClause *Parser::ParseOpenMPOMPXAttributesClause(bool ParseOnly) {
-  SourceLocation Loc = ConsumeToken();
-  // Parse '('.
-  BalancedDelimiterTracker T(*this, tok::l_paren, tok::annot_pragma_openmp_end);
-  if (T.expectAndConsume(diag::err_expected_lparen_after,
-                         getOpenMPClauseName(OMPC_ompx_attribute).data()))
-    return nullptr;
-
-  ParsedAttributes ParsedAttrs(AttrFactory);
-  ParseAttributes(PAKM_GNU | PAKM_CXX11, ParsedAttrs);
-
-  // Parse ')'.
-  if (T.consumeClose())
-    return nullptr;
-
-  if (ParseOnly)
-    return nullptr;
-
-  SmallVector<Attr *> Attrs;
-  for (const ParsedAttr &PA : ParsedAttrs) {
-    switch (PA.getKind()) {
-    case ParsedAttr::AT_AMDGPUFlatWorkGroupSize:
-      if (!PA.checkExactlyNumArgs(Actions, 2))
-        continue;
-      if (auto *A = Actions.CreateAMDGPUFlatWorkGroupSizeAttr(
-              PA, PA.getArgAsExpr(0), PA.getArgAsExpr(1)))
-        Attrs.push_back(A);
-      continue;
-    case ParsedAttr::AT_AMDGPUWavesPerEU:
-      if (!PA.checkAtLeastNumArgs(Actions, 1) ||
-          !PA.checkAtMostNumArgs(Actions, 2))
-        continue;
-      if (auto *A = Actions.CreateAMDGPUWavesPerEUAttr(
-              PA, PA.getArgAsExpr(0),
-              PA.getNumArgs() > 1 ? PA.getArgAsExpr(1) : nullptr))
-        Attrs.push_back(A);
-      continue;
-    case ParsedAttr::AT_CUDALaunchBounds:
-      if (!PA.checkAtLeastNumArgs(Actions, 1) ||
-          !PA.checkAtMostNumArgs(Actions, 2))
-        continue;
-      if (auto *A = Actions.CreateLaunchBoundsAttr(
-              PA, PA.getArgAsExpr(0),
-              PA.getNumArgs() > 1 ? PA.getArgAsExpr(1) : nullptr))
-        Attrs.push_back(A);
-      continue;
-    default:
-      Diag(Loc, diag::warn_omp_invalid_attribute_for_ompx_attributes) << PA;
-      continue;
-    };
-  }
-
-  return Actions.ActOnOpenMPXAttributeClause(Attrs, Loc, T.getOpenLocation(),
-                                             T.getCloseLocation());
-}
-
 /// Parsing of simple OpenMP clauses like 'default' or 'proc_bind'.
 ///
 ///    default-clause: