[PATCH] D69785: [OpenMP] Introduce the OpenMP-IR-Builder

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 4 09:31:54 PST 2019


ABataev added inline comments.


================
Comment at: llvm/include/llvm/IR/OpenMPIRBuilder.h:33
+  /// IDs for all OpenMP directives.
+  enum DirektiveKind {
+#define OMP_DIRECTIVE(Enum, ...) Enum,
----------------
jdoerfert wrote:
> ABataev wrote:
> > 1. `DirectiveKind`
> > 2. Is it possible to merge these 2 types in LLVM and in clang into 1?
> 1. Agreed.
> 2. Yes! Clang needs to eventually use the one here. I can already make that change now or we wait a bit longer and have both coexist. The change would probably be something along the lines of: `using OpenMPDirectiveKind = llvm::OpenMPIRBuilder::DirectiveKind` in clang instead of the enum definition. We probably also want to include `llvm/IR/OpenMPKinds.def` where `OPENMP_DIRECTIVE_EXT` is now used. 
Maybe introduce `llvm/IR/OpenMPKinds.def` and make clang use it in a separate NFC patch?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69785





More information about the cfe-commits mailing list