[llvm] 4cc5145 - [NFCI][CMake] add space among flag sets in all_linker_flags_uppercase

Yuanfang Chen via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 14 18:54:19 PST 2022


Author: Yuanfang Chen
Date: 2022-01-14T18:53:58-08:00
New Revision: 4cc514579f467db51907567b305f21b0607b27d1

URL: https://github.com/llvm/llvm-project/commit/4cc514579f467db51907567b305f21b0607b27d1
DIFF: https://github.com/llvm/llvm-project/commit/4cc514579f467db51907567b305f21b0607b27d1.diff

LOG: [NFCI][CMake] add space among flag sets in all_linker_flags_uppercase

Follow-up on 74bb4ad5d4eb0. It should not change behaviors but a good thing to
do.

Added: 
    

Modified: 
    llvm/cmake/modules/HandleLLVMOptions.cmake

Removed: 
    


################################################################################
diff  --git a/llvm/cmake/modules/HandleLLVMOptions.cmake b/llvm/cmake/modules/HandleLLVMOptions.cmake
index 600d3cc03d58..81f4f6c9de1a 100644
--- a/llvm/cmake/modules/HandleLLVMOptions.cmake
+++ b/llvm/cmake/modules/HandleLLVMOptions.cmake
@@ -490,11 +490,11 @@ if( MSVC )
 
   # Get all linker flags in upper case form so we can search them.
   string(CONCAT all_linker_flags_uppercase
-     ${CMAKE_EXE_LINKER_FLAGS_${uppercase_CMAKE_BUILD_TYPE}}
-     ${CMAKE_EXE_LINKER_FLAGS}
-     ${CMAKE_MODULE_LINKER_FLAGS_${uppercase_CMAKE_BUILD_TYPE}}
-     ${CMAKE_MODULE_LINKER_FLAGS}
-     ${CMAKE_SHARED_LINKER_FLAGS_${uppercase_CMAKE_BUILD_TYPE}}
+     ${CMAKE_EXE_LINKER_FLAGS_${uppercase_CMAKE_BUILD_TYPE}} " "
+     ${CMAKE_EXE_LINKER_FLAGS} " "
+     ${CMAKE_MODULE_LINKER_FLAGS_${uppercase_CMAKE_BUILD_TYPE}} " "
+     ${CMAKE_MODULE_LINKER_FLAGS} " "
+     ${CMAKE_SHARED_LINKER_FLAGS_${uppercase_CMAKE_BUILD_TYPE}} " "
      ${CMAKE_SHARED_LINKER_FLAGS})
   string(TOUPPER "${all_linker_flags_uppercase}" all_linker_flags_uppercase)
 


        


More information about the llvm-commits mailing list