[Openmp-commits] [PATCH] D101977: [OpenMP] Create custom state machines for generic target regions

[Giorgis Georgakoudis via Phabricator via Openmp-commits]

ggeorgakoudis added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:494
+      return false;
+    return ParallelRegions.size() == RHS.ParallelRegions.size();
+  }
----------------
Is equality comparison enough for ParallelRegions or should we change ParallelRegions refer to same function? I guess size equality should be enough since we only add ParallelRegions, but still checking.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101977