[clang-tools-extra] [llvm] [compiler-rt] [clang] [flang] [Flang][OpenMP] Push genEval closer to leaf lowering functions (PR #77760)

via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 16 08:02:18 PST 2024


================
@@ -3189,13 +3208,15 @@ static void genOMP(Fortran::lower::AbstractConverter &converter,
     if ((llvm::omp::allTargetSet & llvm::omp::loopConstructSet)
             .test(ompDirective)) {
       validDirective = true;
-      genTargetOp(converter, eval, semanticsContext, currentLocation,
-                  loopOpClauseList, ompDirective, /*outerCombined=*/true);
+      genTargetOp(converter, eval, semanticsContext, /*genNested=*/false,
----------------
NimishMishra wrote:

`target` has a structured nested block. Can this nested block have additional OpenMP constructs? And if `genNested=false`, then are we relying on the bridge to lower those nested evaluations?

https://github.com/llvm/llvm-project/pull/77760


More information about the llvm-commits mailing list