Index: test/ThinLTO/X86/prefix_replace.ll =================================================================== --- test/ThinLTO/X86/prefix_replace.ll +++ test/ThinLTO/X86/prefix_replace.ll @@ -1,6 +1,5 @@ ; Check that changing the output path via prefix-replace works ; Use of '/' in paths created here make this unsuitable for Windows. -; REQUIRES: shell ; RUN: mkdir -p %T/oldpath ; RUN: opt -module-summary %s -o %T/oldpath/prefix_replace.o ; Ensure that there is no existing file at the new path, so we properly @@ -8,7 +7,7 @@ ; RUN: rm -f %T/newpath/prefix_replace.o.thinlto.bc ; RUN: llvm-lto -thinlto-action=thinlink -o %t.index.bc %T/oldpath/prefix_replace.o -; RUN: llvm-lto -thinlto-action=distributedindexes -thinlto-prefix-replace="%T/oldpath/:%T/newpath/" -thinlto-index %t.index.bc %T/oldpath/prefix_replace.o +; RUN: llvm-lto -thinlto-action=distributedindexes -thinlto-prefix-replace="%T/oldpath/;%T/newpath/" -thinlto-index %t.index.bc %T/oldpath/prefix_replace.o ; RUN: ls %T/newpath/prefix_replace.o.thinlto.bc Index: tools/gold/gold-plugin.cpp =================================================================== --- tools/gold/gold-plugin.cpp +++ tools/gold/gold-plugin.cpp @@ -234,8 +234,8 @@ thinlto_emit_imports_files = true; } else if (opt.startswith("thinlto-prefix-replace=")) { thinlto_prefix_replace = opt.substr(strlen("thinlto-prefix-replace=")); - if (thinlto_prefix_replace.find(":") == std::string::npos) - message(LDPL_FATAL, "thinlto-prefix-replace expects 'old:new' format"); + if (thinlto_prefix_replace.find(";") == std::string::npos) + message(LDPL_FATAL, "thinlto-prefix-replace expects 'old;new' format"); } else if (opt.size() == 2 && opt[0] == 'O') { if (opt[1] < '0' || opt[1] > '3') message(LDPL_FATAL, "Optimization level must be between 0 and 3"); Index: tools/llvm-lto/llvm-lto.cpp =================================================================== --- tools/llvm-lto/llvm-lto.cpp +++ tools/llvm-lto/llvm-lto.cpp @@ -307,9 +307,9 @@ static void getThinLTOOldAndNewPrefix(std::string &OldPrefix, std::string &NewPrefix) { assert(ThinLTOPrefixReplace.empty() || - ThinLTOPrefixReplace.find(":") != StringRef::npos); + ThinLTOPrefixReplace.find(";") != StringRef::npos); StringRef PrefixReplace = ThinLTOPrefixReplace; - std::pair Split = PrefixReplace.split(":"); + std::pair Split = PrefixReplace.split(";"); OldPrefix = Split.first.str(); NewPrefix = Split.second.str(); }