[PATCH] D157418: [WIP][Draft] Changes in OpenMPIRBuilder

Dominik Adamski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 8 10:14:39 PDT 2023


domada created this revision.
domada added reviewers: jdoerfert, jsjodin, dpalermo, agozillon, skatrak.
domada added a project: OpenMP.
Herald added subscribers: ormris, guansong, hiraditya, yaxunl.
Herald added a project: All.
domada requested review of this revision.
Herald added subscribers: llvm-commits, jplehr, sstefan1.
Herald added a project: LLVM.

Original source of the changes: https://github.com/jdoerfert/llvm-project/commit/12137687ee86e827f61ea8f1bfa98b050a7b3cae#diff-a6c8db9d350ec59f4eb93f27f29468b01c9590426a11c9cb79e499bc96b28adc

Rebase assumptions taken by me (Dominik Adamski):

1. Patch depends on: https://reviews.llvm.org/D156423
2. Johannes version of create parallel should be used only for offload pragmas
3. The code generated by OMPIRBuilder::createParallel depends on the OMPIRBuilder config. If OpenMPIRBuilder is configured to generate GPU code then it will generate GPU parallel version of the parallel pragma
4. Johannes changes in llvm/lib/Transforms/Utils/CodeExtractor.cpp are upstreamed

Status of the draft:

1. No additional tests
2. Not finished: createTargetWorkshareLoop


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D157418

Files:
  llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h
  llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
  llvm/lib/Transforms/IPO/OpenMPOpt.cpp
  llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D157418.548269.patch
Type: text/x-patch
Size: 23152 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230808/829839cc/attachment.bin>


More information about the llvm-commits mailing list