[PATCH] D83583: [openmp] Remove OMPConstants.cpp and replace it by OMP.cpp generated by tablegen

[Valentin Clement via Phabricator via llvm-commits]

clementval marked an inline comment as done.
clementval added inline comments.


================
Comment at: llvm/lib/Frontend/OpenMP/CMakeLists.txt:2
+set(LLVM_TARGET_DEFINITIONS ${LLVM_MAIN_INCLUDE_DIR}/llvm/Frontend/OpenMP/OMP.td)
+tablegen(LLVM OMP.cpp --gen-directive-impl)
+add_public_tablegen_target(omp_cpp)
----------------
The reason to have renamed `OMP.cpp.inc` to `OMP.cpp` is that now it is directly source of a library and the name `OMP.cpp.inc` will be used to generated code pieces that have to be included in clang/flang. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83583