<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Nov 29, 2016 at 10:18 AM, Mehdi AMINI via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">mehdi_amini added a comment.<br>
<br>
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.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D27159" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D27159</a><br>
<br>
<br>
<br>
</blockquote></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">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.</div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature">- Michael Spencer</div></div></div><div class="gmail_extra"><br></div></div>