[PATCH] D83635: [OpenMPOpt][WIP] Merge parallel regions

[Giorgis Georgakoudis via Phabricator via llvm-commits]

ggeorgakoudis marked 3 inline comments as done.
ggeorgakoudis added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:42
 
+static cl::opt<bool> EnableParallelRegionMerging(
+    "openmp-opt-enable-merging", cl::ZeroOrMore,
----------------
Option to enable parallel region merging


================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:603
+    // Check if there any __kmpc_push_proc_bind calls for explicit affinities.
+    OMPInformationCache::RuntimeFunctionInfo &ProcBindRFI =
+        OMPInfoCache.RFIs[OMPRTL___kmpc_push_proc_bind];
----------------
Check for explicit proc_bind affinity


================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:650
+
+      auto Remark = [&](OptimizationRemark OR) {
+        OR << "Parallel region at "
----------------
Updated Remark


================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:741
+
+        auto Remark = [&](OptimizationRemark OR) {
+          return OR << "Parallel region at "
----------------
Updated Remark


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83635