[PATCH] D142569: [WIP][OpenMP] Introduce kernel argument
Johannes Doerfert via Phabricator via cfe-commits
cfe-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 cfe-commits
mailing list