forked from OSchip/llvm-project
[CMake] Don't use comma as an alternate separator
Using comma can break in cases when we're passing flags that already use comma as a separator. Fixes PR35504. Differential Revision: https://reviews.llvm.org/D40762 llvm-svn: 319720
This commit is contained in:
parent
6851c0c38e
commit
5a34c34858
|
@ -655,7 +655,7 @@ if (CLANG_ENABLE_BOOTSTRAP)
|
|||
foreach(variableName ${variableNames})
|
||||
if(variableName MATCHES "^BOOTSTRAP_")
|
||||
string(SUBSTRING ${variableName} 10 -1 varName)
|
||||
string(REPLACE ";" "," value "${${variableName}}")
|
||||
string(REPLACE ";" "|" value "${${variableName}}")
|
||||
list(APPEND PASSTHROUGH_VARIABLES
|
||||
-D${varName}=${value})
|
||||
endif()
|
||||
|
@ -671,7 +671,7 @@ if (CLANG_ENABLE_BOOTSTRAP)
|
|||
if("${${variableName}}" STREQUAL "")
|
||||
set(value "")
|
||||
else()
|
||||
string(REPLACE ";" "," value "${${variableName}}")
|
||||
string(REPLACE ";" "|" value "${${variableName}}")
|
||||
endif()
|
||||
list(APPEND PASSTHROUGH_VARIABLES
|
||||
-D${variableName}=${value})
|
||||
|
@ -699,7 +699,7 @@ if (CLANG_ENABLE_BOOTSTRAP)
|
|||
USES_TERMINAL_CONFIGURE 1
|
||||
USES_TERMINAL_BUILD 1
|
||||
USES_TERMINAL_INSTALL 1
|
||||
LIST_SEPARATOR ,
|
||||
LIST_SEPARATOR |
|
||||
)
|
||||
|
||||
# exclude really-install from main target
|
||||
|
|
Loading…
Reference in New Issue