<div dir="ltr"><div class="gmail_extra"><div><div class="gmail_signature">On Thu, May 4, 2017 at 11:41 AM, Rui Ueyama via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br></div></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">ruiu added a comment.<br>
<br>
If we are going to examine this file, I strongly recommend porting the same functionalities (i.e. parallel_for and parallel_for_each) on top of llvm/Support/ThreadPool.h. I don't think that is hard to do as TheadPool.h seems to provide the same functionality as this file. That would be easier than review this file again.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D32826" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D32826</a><br>
<br>
<br>
<br></blockquote><div><br></div><div>My issue with just using Support/ThreadPool for everything is that lld/Parallel.h uses ConcRT on Windows and is quite a bit faster than both our implementations of thread pools. I don't want to punish Windows performance. I'd also rather have our interface match whatever the standard is doing in terms of parallel task execution.</div><div><br></div></div>- Michael Spencer<br></div></div>