[PATCH] D132074: [OpenMP] Add option to assert no nested OpenMP parallelism on the GPU

Joseph Huber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 17 14:47:50 PDT 2022


jhuber6 created this revision.
jhuber6 added reviewers: tianshilei1992, JonChesterfield, jdoerfert, ronlieb, carlo.bertolli, gregrodgers.
Herald added subscribers: guansong, yaxunl.
Herald added a project: All.
jhuber6 requested review of this revision.
Herald added subscribers: openmp-commits, cfe-commits, sstefan1, MaskRay.
Herald added projects: clang, OpenMP.

The OpenMP device runtime needs to support the OpenMP standard. However
constructs like nested parallelism are very uncommon in real application
yet lead to complexity in the runtime that is sometimes difficult to
optimize out. As a stop-gap for performance we should supply an argument
that selectively disables this feature. This patch adds the
`-fopenmp-assume-no-nested-parallelism` argument which explicitly
disables the usee of nested parallelism in OpenMP.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D132074

Files:
  clang/include/clang/Basic/LangOptions.def
  clang/include/clang/Driver/Options.td
  clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp
  clang/lib/Driver/ToolChains/Clang.cpp
  clang/test/OpenMP/target_globals_codegen.cpp
  openmp/libomptarget/DeviceRTL/include/Configuration.h
  openmp/libomptarget/DeviceRTL/src/Configuration.cpp
  openmp/libomptarget/DeviceRTL/src/Parallelism.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D132074.453439.patch
Type: text/x-patch
Size: 12228 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220817/02a3a139/attachment.bin>


More information about the cfe-commits mailing list