[PATCH] D141010: OpenMPOpt: Check nested parallelism in target region

Jose Manuel Monsalve Diaz via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 4 12:26:49 PST 2023


josemonsalve2 added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:4369
+            !FnAA.ReachedUnknownParallelRegions.empty();
+        if (mayContainParRegion)
+          NestedParallelism = true;
----------------
Any reason you want to keep this variable? Why not add the condition directly into the ID?


Repository:
  rOMP OpenMP

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

https://reviews.llvm.org/D141010



More information about the llvm-commits mailing list