win: Move Parallel.h off concrt to cross-platform code
authorNico Weber <nicolasweber@gmx.de>
Thu, 10 Oct 2019 18:57:23 +0000 (18:57 +0000)
committerNico Weber <nicolasweber@gmx.de>
Thu, 10 Oct 2019 18:57:23 +0000 (18:57 +0000)
commitd49600320598a2b4e998e99f714c2e19d95197e8
tree54e865120a81c3ce29e414fab18ab84c7a573638
parent13bd3ef40d8b1586f26a022e01b21e56c91e05bd
win: Move Parallel.h off concrt to cross-platform code

r179397 added Parallel.h and implemented it terms of concrt in 2013.

In 2015, a cross-platform implementation of the functions has appeared
and is in use everywhere but on Windows (r232419).  r246219 hints that
<thread> had issues in MSVC2013, but r296906 suggests they've been fixed
now that we require 2015+.

So remove the concrt code. It's less code, and it sounds like concrt has
conceptual and performance issues, see PR41198.

I built blink_core.dll in a debug component build with full symbols and
in a release component build without any symbols.  I couldn't measure a
performance difference for linking blink_core.dll before and after this
patch.

Differential Revision: https://reviews.llvm.org/D68820

llvm-svn: 374421
llvm/include/llvm/Support/Parallel.h
llvm/lib/Support/Parallel.cpp