[libclc] Quote addition of CLC/LLAsm flags

[nikic]

Summary

Otherwise cmake will insert a semicolon if flags are already set.

[Ericson2314]

Might it be better to stort as a CMake list internally and then convert at only when used?

[nikic]

In D131490#3759520, @Ericson2314 wrote:

Might it be better to stort as a CMake list internally and then convert at only when used?

Is that typically done for CFLAGS-style variables? Seems a bit odd to me to not keep these literal.

[jvesely]

This patch breaks formatting. If there's a spaces after opening "(" there should be one before closing ")"

libclc/CMakeLists.txt

 string( REPLACE "." ";" LLVM_VERSION_LIST ${LLVM_VERSION} )
 list( GET LLVM_VERSION_LIST 0 LLVM_MAJOR )
 list( GET LLVM_VERSION_LIST 1 LLVM_MINOR )
 set( LLVM_VERSION_DEFINE "-DHAVE_LLVM=0x${LLVM_MAJOR}0${LLVM_MINOR}" )
 
 
 # LLVM 13 enables standard includes by default
 if( ${LLVM_VERSION} VERSION_GREATER "12.99.99" )
-        set( CMAKE_LLAsm_FLAGS ${CMAKE_LLAsm_FLAGS} -cl-no-stdinc )
-        set( CMAKE_CLC_FLAGS ${CMAKE_CLC_FLAGS} -cl-no-stdinc )
+        set( CMAKE_LLAsm_FLAGS "${CMAKE_LLAsm_FLAGS} -cl-no-stdinc")
+        set( CMAKE_CLC_FLAGS "${CMAKE_CLC_FLAGS} -cl-no-stdinc")
 endif()
 
 enable_language( CLC LLAsm )
 
 # This needs to be set before any target that needs it
 link_directories( ${LLVM_LIBDIR} )
 
 # Setup prepare_builtins tools