[PATCH] D79390: [Support] Sink LLD's parallel algorithm wrappers to support

Alexandre Ganea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 5 12:26:18 PDT 2020


aganea added a comment.

If the goal is to replace `Parallel.h` with pSTL sometime in the future, then maybe it's better to keep the STL-like API and the policy (but I wouldn't favor that ATM)
I find it pleasant to read the way it is -- if you want to protect from someone doing `using llvm::parallel`, why not keep the LLD nomenclature? `llvm::parallelSort(...)`. It will not be ambiguous, even if you do `using llvm; parallelSort(...);`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79390





More information about the llvm-commits mailing list