diff options
author | Brad King <brad.king@kitware.com> | 2016-07-20 18:39:38 (GMT) |
---|---|---|
committer | Brad King <brad.king@kitware.com> | 2016-07-20 19:00:56 (GMT) |
commit | 34ba5c53481e7f2101dafa735504cb98f94ec6db (patch) | |
tree | e049ca0ea0dc098bcf75ad0dfb2ead07f2ae0c68 /Source/cmMakefileExecutableTargetGenerator.cxx | |
parent | df14a98e9c4af316cd5e75d6af8cc7b75da2db8f (diff) | |
download | CMake-34ba5c53481e7f2101dafa735504cb98f94ec6db.zip CMake-34ba5c53481e7f2101dafa735504cb98f94ec6db.tar.gz CMake-34ba5c53481e7f2101dafa735504cb98f94ec6db.tar.bz2 |
Makefile: Factor out response file checks into common helper
Factor CMAKE_<LANG>_USE_RESPONSE_FILE_FOR_{OBJECTS,LIBRARIES} lookup out
into a common helper. Use a separate helper for each because more
specific logic may be added to each later.
Diffstat (limited to 'Source/cmMakefileExecutableTargetGenerator.cxx')
-rw-r--r-- | Source/cmMakefileExecutableTargetGenerator.cxx | 25 |
1 files changed, 4 insertions, 21 deletions
diff --git a/Source/cmMakefileExecutableTargetGenerator.cxx b/Source/cmMakefileExecutableTargetGenerator.cxx index 8730ccd..3b8bf5a 100644 --- a/Source/cmMakefileExecutableTargetGenerator.cxx +++ b/Source/cmMakefileExecutableTargetGenerator.cxx @@ -277,27 +277,10 @@ void cmMakefileExecutableTargetGenerator::WriteExecutableRule(bool relink) } } - // Select whether to use a response file for objects. - bool useResponseFileForObjects = false; - { - std::string responseVar = "CMAKE_"; - responseVar += linkLanguage; - responseVar += "_USE_RESPONSE_FILE_FOR_OBJECTS"; - if (this->Makefile->IsOn(responseVar)) { - useResponseFileForObjects = true; - } - } - - // Select whether to use a response file for libraries. - bool useResponseFileForLibs = false; - { - std::string responseVar = "CMAKE_"; - responseVar += linkLanguage; - responseVar += "_USE_RESPONSE_FILE_FOR_LIBRARIES"; - if (this->Makefile->IsOn(responseVar)) { - useResponseFileForLibs = true; - } - } + bool useResponseFileForObjects = + this->CheckUseResponseFileForObjects(linkLanguage); + bool const useResponseFileForLibs = + this->CheckUseResponseFileForLibraries(linkLanguage); // Expand the rule variables. { |