jdoerfert marked 2 inline comments as done. jdoerfert added inline comments.
================ Comment at: llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp:241 + // This seems to be used only once without much change of reuse, could live in + // OMPKinds.def but seems not necessary. + Value *CancelKind = nullptr; ---------------- JonChesterfield wrote: > The integer numbers correspond to the `kmp_cancel_kind_t` enum in > `runtime/src/kmp.h`. The target offloading presently ignores this argument, > but the host version has a control flow dependency on it. > > I think the enum should be shared between the compiler and the runtime, or > failing that, some test code should include kmp.h and check the numbers still > match. > > This feels like a familiar point - I've just sent an email to openmp-dev to > discuss whether we can share constants between the two without copy & paste. I move them into OMPKinds.def, we can share them from there. I'll respond to the mailing list. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71948/new/ https://reviews.llvm.org/D71948 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
