<div dir="ltr">I know that Cygwin doesn't have C++11 STL threading headers:<div><a href="https://llvm.org/bugs/show_bug.cgi?id=25290">https://llvm.org/bugs/show_bug.cgi?id=25290</a><br></div><div><br></div><div>I've also heard that some Power platforms don't, but I don't have any reference for that.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 16, 2015 at 8:24 PM, Mehdi Amini <span dir="ltr"><<a href="mailto:mehdi.amini@apple.com" target="_blank">mehdi.amini@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div><br><br>Sent from my iPhone</div><div><div class="h5"><div><br>On Dec 15, 2015, at 12:05 PM, Reid Kleckner <<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>> wrote:<br><br></div><blockquote type="cite"><div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Dec 15, 2015 at 1:55 PM, Mehdi Amini <span dir="ltr"><<a href="mailto:mehdi.amini@apple.com" target="_blank">mehdi.amini@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="auto"><div><br><br>Sent from my iPhone</div><span><div><br>On Dec 15, 2015, at 11:50 AM, Reid Kleckner <<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>> wrote:<br><br></div><blockquote type="cite"><div><div dir="ltr">I think I fixed that const qualified issue with r255678. I still have warnings from ppltasks.h, which is this connect issue:<div><a href="https://connect.microsoft.com/VisualStudio/feedback/details/1132688/ppltasks-h-does-not-compile-cleanly" target="_blank">https://connect.microsoft.com/VisualStudio/feedback/details/1132688/ppltasks-h-does-not-compile-cleanly</a><br></div><div><br></div><div>I think we should ban inclusions of <future> in LLVM and make a wrapper header that uses pragmas to disable warnings in <future> transitive includes.</div></div></div></blockquote><div><br></div></span><div>#include "llvm/Support/future.h"?</div><div>I can give a try at that later today, but since I can't validate with MSVC it might take a few tentatives, so I don't mind if someone else do it.</div></div></blockquote><div><br></div><div>I'm actually inclined to throw them all in "llvm/Support/thread.h", since we're probably going to need more ifdefs to avoid these includes when !LLVM_ENABLE_THREADS.</div><div><br></div><div>Have you considered how ThreadPool should operate when the STL does not provide C++11 threading headers? We told users in that situation to turn off LLVM_ENABLE_THREADS, but this would de-support such a configuration altogether.</div></div></div></div>
</div></blockquote><br></div></div><div>My understanding (and the information that I got on IRC) was that the only reason to have LLVM_ENABLE_THREADS is to avoid some performance hit because of the need of locking on some globals we have here and there.</div><div><br></div><div>So I didn't plan to support any<span> C++11 implementation that do not allow std::async with launch deferred. (Note: we never need to include <thread> for that)</span></div><div><span><br></span></div><div><span>If anyone has such need, it will require to rewrite a custom implementation for std::async with launch::deferred that can be replaced like we do in lib/Support/thread.h for std::thread.</span></div><div><span>(I'll be happy to review such patch)</span></div><span class="HOEnZb"><font color="#888888"><div><span><br></span></div><div><span>-- </span></div><div><span>Mehdi</span></div><div><span><br></span></div><div><span><br></span></div></font></span></div></blockquote></div><br></div>