[PATCH] D88706: [OpenMP][MLIR] WIP : Fix for nested parallel region

[Kiran Chandramohan via Phabricator via llvm-commits]

kiranchandramohan created this revision.
kiranchandramohan added reviewers: SouraVX, jdoerfert, kiranktp.
Herald added subscribers: llvm-commits, tatianashp, msifontes, jurahul, Kayjukh, grosul1, Joonsoo, liufengdb, aartbik, lucyrfox, mgester, arpith-jacob, antiagainst, shauheen, rriddle, mehdi_amini, guansong, hiraditya, yaxunl.
Herald added a reviewer: ftynse.
Herald added projects: MLIR, LLVM.
kiranchandramohan requested review of this revision.
Herald added subscribers: sstefan1, stephenneuendorffer, nicolasvasilache.

Fix for nested parallel regions. Does the following,

1. Switch to OpenMPIRBuilder version which maintains allocaIP. (An initial version of https://reviews.llvm.org/D82470)
2. Create jump to continuation block by converting the terminator

Note:

1. This patch should be applied after reverting 19756ef53a498b7aa1fbac9e3a7cd3aa8e110fad <https://reviews.llvm.org/rG19756ef53a498b7aa1fbac9e3a7cd3aa8e110fad>.
2. This fix is required in the lowering of Master Op (https://reviews.llvm.org/D87247)


https://reviews.llvm.org/D88706

Files:
  llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h
  llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
  llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp
  mlir/include/mlir/Target/LLVMIR/ModuleTranslation.h
  mlir/lib/Target/LLVMIR/ModuleTranslation.cpp
  mlir/test/Target/openmp-llvm.mlir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88706.295680.patch
Type: text/x-patch
Size: 17546 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201001/e97b6ce5/attachment-0001.bin>