diff options
author | Alexandre Ganea <alexandre.ganea@ubisoft.com> | 2018-12-18 18:17:00 +0000 |
---|---|---|
committer | Alexandre Ganea <alexandre.ganea@ubisoft.com> | 2018-12-18 18:17:00 +0000 |
commit | 62c58803add8efd8283e7966037aa3e9f6148357 (patch) | |
tree | 49050c6528be0ba6169f96f1f5727e2a3c84d00b | |
parent | 5ddfa0b2a6553bbb7f6352d5c9d867dc23aeb65e (diff) |
[CMake] Default options for faster executables on MSVC
- Disable incremental linking by default. /INCREMENTAL adds extra thunks in the EXE, which makes execution slower.
- Set /MT (static CRT lib) by default instead of CMake's default /MD (dll CRT lib). The previous default /MD makes all DLL functions to be thunked, thus making execution slower (memcmp, memset, etc.)
- Adds LLVM_ENABLE_INCREMENTAL_LINK which is set to OFF by default.
Differential revision: https://reviews.llvm.org/D55056
-rw-r--r-- | llvm/CMakeLists.txt | 4 | ||||
-rw-r--r-- | llvm/cmake/modules/ChooseMSVCCRT.cmake | 9 | ||||
-rw-r--r-- | llvm/cmake/modules/HandleLLVMOptions.cmake | 8 |
3 files changed, 21 insertions, 0 deletions
diff --git a/llvm/CMakeLists.txt b/llvm/CMakeLists.txt index e2e35dcebdb..15f95d2e580 100644 --- a/llvm/CMakeLists.txt +++ b/llvm/CMakeLists.txt @@ -370,6 +370,10 @@ option(LLVM_ENABLE_LLD "Use lld as C and C++ linker." OFF) option(LLVM_ENABLE_PEDANTIC "Compile with pedantic enabled." ON) option(LLVM_ENABLE_WERROR "Fail and stop if a warning is triggered." OFF) +if (MSVC) + option(LLVM_ENABLE_INCREMENTAL_LINK "Link incrementally. Enabling it might produce slower executables." OFF) +endif() + option(LLVM_ENABLE_DUMP "Enable dump functions even when assertions are disabled" OFF) if( NOT uppercase_CMAKE_BUILD_TYPE STREQUAL "DEBUG" ) diff --git a/llvm/cmake/modules/ChooseMSVCCRT.cmake b/llvm/cmake/modules/ChooseMSVCCRT.cmake index 0e6e1aa5525..cdd7deec4c8 100644 --- a/llvm/cmake/modules/ChooseMSVCCRT.cmake +++ b/llvm/cmake/modules/ChooseMSVCCRT.cmake @@ -66,6 +66,15 @@ variables (LLVM_USE_CRT_DEBUG, etc) instead.") get_current_crt(LLVM_USE_CRT_${build} MSVC_CRT_REGEX CMAKE_CXX_FLAGS_${build}) + + # Make /MT the default in Release builds to make them faster + # and avoid the DLL function thunking. + if ((${build} STREQUAL "MINSIZEREL") OR + (${build} STREQUAL "RELEASE") OR + (${build} STREQUAL "RELWITHDEBINFO")) + set(LLVM_USE_CRT_${build} "MT") + endif() + set(LLVM_USE_CRT_${build} "${LLVM_USE_CRT_${build}}" CACHE STRING "Specify VC++ CRT to use for ${build_type} configurations." diff --git a/llvm/cmake/modules/HandleLLVMOptions.cmake b/llvm/cmake/modules/HandleLLVMOptions.cmake index 49db3088bbc..e8b0e27e000 100644 --- a/llvm/cmake/modules/HandleLLVMOptions.cmake +++ b/llvm/cmake/modules/HandleLLVMOptions.cmake @@ -381,6 +381,14 @@ if( MSVC ) # "Enforce type conversion rules". append("/Zc:rvalueCast" CMAKE_CXX_FLAGS) + if (CMAKE_CXX_COMPILER_ID MATCHES "MSVC" AND NOT LLVM_ENABLE_INCREMENTAL_LINK) + foreach(CONFIG RELEASE RELWITHDEBINFO MINSIZEREL) + foreach(FLAG EXE MODULE SHARED STATIC) + string(REGEX REPLACE "[-/](INCREMENTAL:YES|INCREMENTAL:NO|INCREMENTAL)" "/INCREMENTAL:NO" CMAKE_${FLAG}_LINKER_FLAGS_${CONFIG} "${CMAKE_${FLAG}_LINKER_FLAGS_${CONFIG}}") + endforeach() + endforeach() + endif() + if (CMAKE_CXX_COMPILER_ID MATCHES "Clang" AND NOT LLVM_ENABLE_LTO) # clang-cl and cl by default produce non-deterministic binaries because # link.exe /incremental requires a timestamp in the .obj file. clang-cl |