[PATCH] D69930: [OpenMP] Introduce the OpenMPOpt transformation pass

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:144-145
+  bool deduplicateRuntimeCalls(Function &F, RuntimeFunctionInfo &RFI,
+                               Value *ReplVal = nullptr) {
+    auto &Uses = RFI.UsesMap[&F];
+    if (Uses.size() + (ReplVal != nullptr) < 2)
----------------
jdoerfert wrote:
> lebedev.ri wrote:
> > Should there be a debug counter controlled early exit as the first line of function?
> I don't understand what you want here, could you explain?
http://llvm.org/docs/ProgrammersManual.html#adding-debug-counters-to-aid-in-debugging-your-code


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69930