[PATCH] D84347: [MLIR,OpenMP] Lowering of parallel operation: proc_bind clause 2/n

[Kiran Chandramohan via Phabricator via llvm-commits]

kiranchandramohan updated this revision to Diff 283567.
kiranchandramohan added a comment.

Addressing @jdoerfert's review comment.

The values that can be specified for the proc_bind clause are specified in the OMP.td tablegen file in the llvm/Frontend/OpenMP directory. From this single source of truth enumeration for proc_bind is generated in llvm and mlir (used in specification of the parallel Operation in the OpenMP dialect). A function to return the enum value from the string representation is also generated.


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

https://reviews.llvm.org/D84347

Files:
  llvm/include/llvm/Frontend/Directive/DirectiveBase.td
  llvm/include/llvm/Frontend/OpenMP/OMP.td
  llvm/include/llvm/Frontend/OpenMP/OMPConstants.h
  llvm/utils/TableGen/DirectiveEmitter.cpp
  mlir/include/mlir/Dialect/OpenMP/CMakeLists.txt
  mlir/include/mlir/Dialect/OpenMP/OpenMPOps.td
  mlir/lib/Target/LLVMIR/ModuleTranslation.cpp
  mlir/test/Target/openmp-llvm.mlir
  mlir/tools/mlir-tblgen/CMakeLists.txt
  mlir/tools/mlir-tblgen/OpenMPCommonGen.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D84347.283567.patch
Type: text/x-patch
Size: 15877 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200806/2bf30d61/attachment.bin>