[libcxx-commits] [pstl] r356194 - [pstl] By default, disable the parallel policies

Louis Dionne via libcxx-commits libcxx-commits at lists.llvm.org
Thu Mar 14 12:33:59 PDT 2019


Author: ldionne
Date: Thu Mar 14 12:33:58 2019
New Revision: 356194

URL: http://llvm.org/viewvc/llvm-project?rev=356194&view=rev
Log:
[pstl] By default, disable the parallel policies

Since we don't have any non-trivial PSTL backend that doesn't require
TBB yet, enabling the parallel policies by default breaks people that
try to build all of LLVM without having an installation of TBB. Since
this is unacceptable, parallel policies are disabled by default.

We can re-enable it once we have a backend that does not require anything
beyond what C++ already requires. For example, we could have a simple
backend that uses std::thread by default or something along those lines,
with the understanding that vendors would use their own (more efficient)
backend.

Modified:
    pstl/trunk/CMakeLists.txt

Modified: pstl/trunk/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/pstl/trunk/CMakeLists.txt?rev=356194&r1=356193&r2=356194&view=diff
==============================================================================
--- pstl/trunk/CMakeLists.txt (original)
+++ pstl/trunk/CMakeLists.txt Thu Mar 14 12:33:58 2019
@@ -16,7 +16,7 @@ math(EXPR VERSION_MINOR "${PARALLELSTL_V
 
 project(ParallelSTL VERSION ${VERSION_MAJOR}.${VERSION_MINOR} LANGUAGES CXX)
 
-option(PARALLELSTL_USE_PARALLEL_POLICIES "Enable parallel policies" ON)
+option(PARALLELSTL_USE_PARALLEL_POLICIES "Enable parallel policies" OFF)
 set(PARALLELSTL_BACKEND "tbb" CACHE STRING "Threading backend; defaults to TBB")
 
 include(CMakePackageConfigHelpers)




More information about the libcxx-commits mailing list