[PATCH] D117510: [Support] Merge parallel_for_each into parallelForEach

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 17 12:33:58 PST 2022


MaskRay added inline comments.


================
Comment at: llvm/include/llvm/Support/Parallel.h:246
+    }
+    for (; Begin != End; ++Begin)
+      Fn(Begin);
----------------
Removing 
```
for (; Begin != End; ++Begin)
  Fn(Begin);
return;
```
is still correct (my lld executable will be 9KiB smaller on top of the current decrease) but make the TaskGroup have less parallelism.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117510



More information about the llvm-commits mailing list