[PATCH] D86071: [MLIR][OpenMP] Add omp.do operation

[Valentin Clement via Phabricator via llvm-commits]

clementval added inline comments.


================
Comment at: llvm/include/llvm/Frontend/OpenMP/OMP.td:120
+
+// static and auto are C++ keywords so need a _ to disambiguate
+def OMP_SCHEDULE_static_ : ClauseVal<"static_",2,1> {}
----------------
kiranchandramohan wrote:
> For the default clause in parallel we have used a prefix "def" to fix this issue. I think we need to standardize this. Would converting the first letter to caps be a reasonable workaround since reserved keywords do not have a letter with caps as first letter?
That's probably a good idea to have a standard way to do that. +1 for the first letter capitalized if it works. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86071