[PATCH] D138217: [OpenMP] Initial parsing and semantic analysis support for 'strict' modifier with 'grainsize' clause of 'taskloop' construct

[Alexey Bataev via Phabricator via cfe-commits]

ABataev added a comment.

1. Add full diff context
2. Add unsuccessful tests for the modifier (parsing and sema)



================
Comment at: clang/lib/Parse/ParseOpenMP.cpp:3811
+    // Parse optional <grainsize modifier> ':'
+    auto Modifier = getOpenMPSimpleClauseType(
+        Kind, Tok.isAnnotation() ? "" : PP.getSpelling(Tok), getLangOpts());
----------------
Use actual type instead of auto here


================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:16812
+  case OMPC_grainsize:
+    assert(Argument.size() == 1 && ArgumentLoc.size() == 1);
+    Res = ActOnOpenMPGrainsizeClause(
----------------
Add a message here


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138217/new/

https://reviews.llvm.org/D138217