[PATCH] D89861: [Flang][OpenMP 4.5] Add semantic check for OpenMP default clause

[Valentin Clement via Phabricator via llvm-commits]

clementval requested changes to this revision.
clementval added inline comments.
This revision now requires changes to proceed.


================
Comment at: llvm/include/llvm/Frontend/OpenMP/OMP.td:309
   let allowedOnceClauses = [
+    VersionedClause<OMPC_Default>,
     VersionedClause<OMPC_If>,
----------------
Please remove the occurrence in `allowedClauses` if you move it here. 


================
Comment at: llvm/include/llvm/Frontend/OpenMP/OMP.td:330
   let allowedOnceClauses = [
+    VersionedClause<OMPC_Default>,
     VersionedClause<OMPC_If>,
----------------
Please remove the occurrence in `allowedClauses` if you move it here. 


================
Comment at: llvm/include/llvm/Frontend/OpenMP/OMP.td:821
   let allowedOnceClauses = [
+    VersionedClause<OMPC_Default>,
     VersionedClause<OMPC_If>,
----------------
Please remove the occurrence in `allowedClauses` if you move it here. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89861