[PATCH] D27159: [ThreadPool] Simplify the interface

Michael Spencer via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 28 18:04:44 PST 2016


On Tue, Nov 29, 2016 at 10:18 AM, Mehdi AMINI via Phabricator <
reviews at reviews.llvm.org> wrote:

> mehdi_amini added a comment.
>
> What is the motivation? This is removing a feature. Even though it is not
> used at that time, it is tested and could find its use in the future.
>
>
> https://reviews.llvm.org/D27159
>
>
>
>
shared_future has a large performance overhead compared to just handing off
a function pointer to another thread to run. Providing that as the default
interface isn't great, and it's trivial to build that interface on top of
the other. The reason to do this now is we'd like to reduce the differences
between the llvm and lld thread pool so they can be merged.

- Michael Spencer
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161129/ba6566d0/attachment.html>


More information about the llvm-commits mailing list