Skip to content

Commit 271c128

Browse files
James BarbettiJames Barbetti
James Barbetti
authored and
James Barbetti
committed
Doesn't set /MP in Visual Studio builds (that's now breaking in VS2019?!).
1 parent 4715234 commit 271c128

File tree

1 file changed

+1
-8
lines changed

1 file changed

+1
-8
lines changed

CMakeLists.txt

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -315,17 +315,10 @@ if (MSVC)
315315
add_definitions(/D_UWIN)
316316
set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} /GX")
317317
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} /I${EIGEN3_INCLUDE_DIR} /GX")
318-
else()
319-
#enable multi-processor compilation
320-
add_definitions("/MP")
321-
#Though zlib mightn't compile with it set, with Visual Studio 2019.
322-
#It didn't for me. But it compiled with it with Visual Studio 2022.
323318
endif()
324319
if (CMAKE_BUILD_TYPE STREQUAL "Release")
325320
add_definitions(/Ot /Oi)
326-
if (VCC)
327-
#add_definitions(/O2) - not compatible with /RTC1
328-
elseif (ICC)
321+
if (ICC)
329322
add_definitions(/O3)
330323
endif()
331324
endif()

0 commit comments

Comments
 (0)