[PATCH] D135695: [OMPIRBuilder] Support depend clause for task construct

[Thorsten via Phabricator via llvm-commits]

tschuett added inline comments.


================
Comment at: llvm/include/llvm/Frontend/OpenMP/OMPConstants.h:210
 
+enum OMPRTLDependenceKindTy {
+  OMPDepIn = 0x01,
----------------
The enums above are `enum class`.


================
Comment at: llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h:643
 
+  enum OpenMPDependKind {
+    OMP_DEPEND_in,
----------------
Why is this not `enum class`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135695