jdoerfert added a comment. I think the rest looks good.
================ Comment at: llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h:213 + enum class OMPOrderClauseValue { NotSpecified, Concurrent }; + ---------------- I think this belongs into `llvm/include/llvm/Frontend/OpenMP/OMPConstants.h` either explicitly or via the `llvm/Frontend/OpenMP/OMPKinds.def` logic. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134046/new/ https://reviews.llvm.org/D134046 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits