[PATCH] D72304: [OpenMP]{OMPIRBuilder] Add Directives (master and critical) to OMPBuilder.
Fady Ghanim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 15 09:36:02 PST 2020
fghanim updated this revision to Diff 238301.
fghanim marked 25 inline comments as done.
fghanim added a comment.
Addressing reviewer comments
- Added regression tests
- styling
- adding asserts and todo where needed.
Also, Now `EmitInlinedRegion` will merge blocks where legal.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72304/new/
https://reviews.llvm.org/D72304
Files:
clang/lib/CodeGen/CGStmtOpenMP.cpp
clang/test/OpenMP/critical_codegen.cpp
clang/test/OpenMP/master_codegen.cpp
llvm/include/llvm/Frontend/OpenMP/OMPConstants.h
llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h
llvm/include/llvm/Frontend/OpenMP/OMPKinds.def
llvm/lib/Frontend/OpenMP/OMPConstants.cpp
llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D72304.238301.patch
Type: text/x-patch
Size: 45698 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200115/2bf5e0f3/attachment-0001.bin>
More information about the llvm-commits
mailing list