[PATCH] D147219: [OpenMP][Flang][MLIR] Lowering of requires directive from MLIR to LLVM IR
Sergio Afonso via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 2 07:06:06 PDT 2023
skatrak updated this revision to Diff 527854.
skatrak added a comment.
Herald added subscribers: cfe-commits, hiraditya.
Herald added a project: clang.
Update and remove OpenMP dialect dependency from the generic LLVM IR translation.
Followed @kiranchandramohan's suggestion in the comments for D147172 <https://reviews.llvm.org/D147172>, since the
current approach prevents the use of OpenMP dialect-specific attributes for
initializing the `OpenMPIRBuilder` configuration. One of these is defined for
representing 'requires' clauses, which need to be stored in the
`OpenMPIRBuilderConfig`, so a different approach is necessary. The approach
implemented in this review is based on the `amendOperation` translation flow.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D147219/new/
https://reviews.llvm.org/D147219
Files:
clang/lib/CodeGen/CGOpenMPRuntime.cpp
llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h
llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp
mlir/lib/Target/LLVMIR/ModuleTranslation.cpp
mlir/test/Target/LLVMIR/openmp-llvm.mlir
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D147219.527854.patch
Type: text/x-patch
Size: 16194 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230602/b8df6e37/attachment.bin>
More information about the llvm-commits
mailing list