[PATCH] D33016: [Parallel] Add support for parallel execution policies to match C++ Parallelism TS

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc added a comment.

Other than a YAGNI argument against the type-erased stuff, seems fine to me.



================
Comment at: lld/include/lld/Core/Parallel.h:54
+
+// Dynamic execution policy
+struct execution_policy {
----------------
This seems like a *lot* of machinery for one use case in one part of LLD.... I'd honestly just not implement this until someone really wants it because they need it....


https://reviews.llvm.org/D33016