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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits