From efb08cd11c4776ad9ef0561fbec735693a650b3f Mon Sep 17 00:00:00 2001 From: Abdullah Tahiri Date: Mon, 22 Jun 2015 05:04:48 +0200 Subject: [PATCH] Sketcher: Disable Eigen multithread =================================== Eigen currently only provides multithread for matrix products. It has been reported that it is even slightly slower than single-thread. Therefore, temporarely the code is commented out. If Eigen ever implements multithread for QR decomposition, then it will make a lot of sense to enable this. --- src/Mod/Sketcher/App/CMakeLists.txt | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/Mod/Sketcher/App/CMakeLists.txt b/src/Mod/Sketcher/App/CMakeLists.txt index 43a7b3c8c..520d8a9ef 100644 --- a/src/Mod/Sketcher/App/CMakeLists.txt +++ b/src/Mod/Sketcher/App/CMakeLists.txt @@ -4,11 +4,12 @@ else(MSVC) add_definitions(-DHAVE_LIMITS_H -DHAVE_CONFIG_H) endif(MSVC) -find_package(OpenMP) -if (OPENMP_FOUND) - set (CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${OpenMP_C_FLAGS}") - set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${OpenMP_CXX_FLAGS}") -endif() +# Uncomment this code if you want to try Eigen's multithread +#find_package(OpenMP) +#if (OPENMP_FOUND) +# set (CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${OpenMP_C_FLAGS}") +# set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${OpenMP_CXX_FLAGS}") +#endif() # set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DEIGEN_NO_DEBUG") # Necessary to avoid SparseQR crash with Eigen 3.2 due to a wrong assertion