aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNaios <naios-dev@live.de>2017-12-23 02:12:45 +0100
committerNaios <naios-dev@live.de>2017-12-23 02:14:05 +0100
commitd3d003d69e91fb9b019a06b0402acebae5d6d250 (patch)
tree522c69c34582e40b22eb06eba4c605eaf036a3bc
parentb08280fcfee500acf8b951ec2c05e0760d4b9955 (diff)
Build: Re-enable the /Zm workaround for MSVC (cotire version)
* Thanks Spec for reporting this issue * See https://community.trinitycore.org/topic/120-compile-or-cmake-problems/?do=findComment&comment=83912 * Ref 697b4dbef20
-rw-r--r--cmake/macros/FindPCHSupport.cmake10
1 files changed, 10 insertions, 0 deletions
diff --git a/cmake/macros/FindPCHSupport.cmake b/cmake/macros/FindPCHSupport.cmake
index abe7f40d171..9c77605616b 100644
--- a/cmake/macros/FindPCHSupport.cmake
+++ b/cmake/macros/FindPCHSupport.cmake
@@ -1,3 +1,13 @@
+if (MSVC)
+ # Specify the maximum PreCompiled Header memory allocation limit
+ # Fixes a compiler-problem when using PCH - the /Ym flag is adjusted by the compiler in MSVC2012,
+ # hence we need to set an upper limit with /Zm to avoid discrepancies)
+ # (And yes, this is a verified, unresolved bug with MSVC... *sigh*)
+ #
+ # Note: This workaround was verified to be required on MSVC 2017 as well
+ set(COTIRE_PCH_MEMORY_SCALING_FACTOR 500)
+endif()
+
include(cotire)
function(ADD_CXX_PCH TARGET_NAME_LIST PCH_HEADER)