[PATCH] D101699: [Support/Parallel] Add a special case for 0/1 items to llvm::parallel_for_each.

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 4 10:49:02 PDT 2021


rnk added a comment.

Change looks good.

In the long run, we should try to build on an existing C++17 parallel algorithms library, such as the pstl in our own monorepo. I don't know how far off it is before we can raise the LLVM toolchain requirements to C++17, and after that, how much platform support will hold us back, but maybe we can hack the pstl into the LLVM build if it becomes a sticking point.

The important thing for LLVM is that we express our code using well-known, deterministic parallel algorithms, and we can pick up better implementations later.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101699



More information about the llvm-commits mailing list