Nico/Viano-Core
1
0 Fork 0
Code Issues Pull Requests Actions 10 Packages Projects Releases Wiki Activity

Build: Fixed VS always triggering a full build when PCH is enabled, even if only a source file was edited

Browse Source 
(cherry picked from commit 9b1dfe5e5e968c6ee3a2682ae2106735daeccbd7)
(cherry picked from commit b64583b7c14da408b34d2c92a02e5529594ef181)
This commit is contained in:
Shauren authored and Ovahlord committed 2020-04-16 13:00:59 +02:00
1 parent ec6d64b9c6
commit ad377241f9
2 files changed
+27 -8
Download Patch File Download Diff File Expand all files Collapse all files

No files matched your search

dep/cotire/CMake/cotire.cmake
+21 -7
View File
@@ -2341,7 +2341,7 @@ function (cotire_setup_pch_file_inclusion _language _target _wholeTarget _prefix
endif()
endfunction()
function (cotire_setup_prefix_file_inclusion _language _target _prefixFile)
function (cotire_setup_prefix_file_inclusion _language _target _prefixFile _prefixFileWasGenerated)
set (_sourceFiles ${ARGN})
# force the inclusion of the prefix header for the given source files
set (_flags "")
@@ -2352,8 +2352,10 @@ function (cotire_setup_prefix_file_inclusion _language _target _prefixFile)
set_property (SOURCE ${_sourceFiles} APPEND_STRING PROPERTY COMPILE_FLAGS " ${_flags} ")
# mark sources as cotired to prevent them from being used in another cotired target
set_source_files_properties(${_sourceFiles} PROPERTIES COTIRE_TARGET "${_target}")
# make object files generated from source files depend on prefix header
set_property (SOURCE ${_sourceFiles} APPEND PROPERTY OBJECT_DEPENDS "${_prefixFile}")
if (_prefixFileWasGenerated)
# make object files generated from source files depend on prefix header
set_property (SOURCE ${_sourceFiles} APPEND PROPERTY OBJECT_DEPENDS "${_prefixFile}")
endif()
endfunction()
function (cotire_get_first_set_property_value _propertyValueVar _type _object)
@@ -2866,9 +2868,21 @@ function (cotire_process_target_language _language _configurations _target _whol
if (_prefixFile)
# check for user provided prefix header files
get_property(_prefixHeaderFiles TARGET ${_target} PROPERTY COTIRE_${_language}_PREFIX_HEADER_INIT)
set (_prefixFileWasGenerated TRUE)
if (_prefixHeaderFiles)
cotire_setup_prefix_generation_from_provided_command(
${_language} ${_target} "${_targetConfigScript}" "${_prefixFile}" _cmds ${_prefixHeaderFiles})
list (LENGTH _prefixHeaderFiles _numberOfprefixHeaderFiles)
# if there is only one header specified in COTIRE_${_language}_PREFIX_HEADER_INIT then use it directly
# this works around a bug/change that causes Visual Studio to always rebuild entire project
# even if only one source file changed
# custom commands in cmake are designed to be always "out of date" so generation always happens
# Visual Studio determines "out of date" status by last modification timestamp, not content
if (_numberOfprefixHeaderFiles GREATER 1 OR NOT CMAKE_${_language}_COMPILER_ID MATCHES "MSVC|Intel")
cotire_setup_prefix_generation_from_provided_command(
${_language} ${_target} "${_targetConfigScript}" "${_prefixFile}" _cmds ${_prefixHeaderFiles})
else()
list (GET _prefixHeaderFiles 0 _prefixFile)
set (_prefixFileWasGenerated FALSE)
endif()
else()
cotire_setup_prefix_generation_from_unity_command(
${_language} ${_target} "${_targetConfigScript}" "${_prefixFile}" "${_unityFile}" _cmds ${_unitySourceFiles})
@@ -2891,7 +2905,7 @@ function (cotire_process_target_language _language _configurations _target _whol
endif()
elseif (_prefixHeaderFiles)
# user provided prefix header must be included unconditionally
cotire_setup_prefix_file_inclusion(${_language} ${_target} "${_prefixFile}" ${_sourceFiles})
cotire_setup_prefix_file_inclusion(${_language} ${_target} "${_prefixFile}" ${_prefixFileWasGenerated} ${_sourceFiles})
endif()
endif()
# mark target as cotired for language
@@ -2995,7 +3009,7 @@ function (cotire_setup_unity_target_pch_usage _languages _target)
get_property(_prefixFile TARGET ${_target} PROPERTY COTIRE_${_language}_PREFIX_HEADER)
if (_userPrefixFile AND _prefixFile)
# user provided prefix header must be included unconditionally by unity sources
cotire_setup_prefix_file_inclusion(${_language} ${_target} "${_prefixFile}" ${_unityFiles})
cotire_setup_prefix_file_inclusion(${_language} ${_target} "${_prefixFile}" TRUE ${_unityFiles})
endif()
endif()
endforeach()