[PATCH] D117510: [Support] Simplify parallelForEach{,N}

[Chris Lattner via Phabricator via llvm-commits]

lattner added inline comments.


================
Comment at: llvm/lib/Support/Parallel.cpp:198
+        for (size_t I = Begin, E = Begin + TaskSize; I != E; ++I)
+          Fn(I);
+      });
----------------
rnk wrote:
> I believe the old code was templated in an effort to avoid having an indirect call in the inner loop. However, I don't think that actually matters. These constructs have so much overhead that they are only useful for coarse-grained parallelism, not vectorizable inner loops.
Right, I completely agree with you


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117510