[PATCH] D142569: [WIP][OpenMP] Introduce kernel argument

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 25 17:28:02 PST 2023


jdoerfert added inline comments.


================
Comment at: openmp/libomptarget/DeviceRTL/src/Configuration.cpp:57
+  return !__omp_rtl_assume_no_nested_parallelism ||
+         state::getKernelEnvironment().Configuration.MayUseNestedParallelism;
 }
----------------
This looks wrong. We want either flag to allow us to remove nested parallelism handling. So 
`__omp_rtl_assume_no_nested_parallelism ` is good enough, and
`!state::getKernelEnvironment().Configuration.MayUseNestedParallelism` is good enough.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142569/new/

https://reviews.llvm.org/D142569



More information about the llvm-commits mailing list