Index: clang/lib/Driver/Driver.cpp =================================================================== --- clang/lib/Driver/Driver.cpp +++ clang/lib/Driver/Driver.cpp @@ -4635,9 +4635,16 @@ const char *Driver::GetNamedOutputPath(Compilation &C, const JobAction &JA, const char *BaseInput, - StringRef BoundArch, bool AtTopLevel, + StringRef OrigBoundArch, bool AtTopLevel, bool MultipleArchs, StringRef OffloadingPrefix) const { + std::string BoundArch = OrigBoundArch.str(); +#if defined(_WIN32) + // BoundArch may contains ':', which is invalid in file names on Windows, + // therefore replace it with '%'. + std::replace(BoundArch.begin(), BoundArch.end(), ':', '%'); +#endif + llvm::PrettyStackTraceString CrashInfo("Computing output path"); // Output to a user requested destination? if (AtTopLevel && !isa(JA) && !isa(JA)) { Index: clang/test/Driver/hip-windows-filename.hip =================================================================== --- /dev/null +++ clang/test/Driver/hip-windows-filename.hip @@ -0,0 +1,10 @@ +// REQUIRES: system-windows, clang-driver, amdgpu-registered-target + +// RUN: %clang -### -target x86_64-pc-windows-msvc \ +// RUN: -x hip \ +// RUN: --offload-arch=gfx908:xnack+ \ +// RUN: -nogpuinc -nogpulib -save-temps \ +// RUN: %s 2>&1 | FileCheck %s + +// CHECK: "-o" "hip-windows-filename-hip-amdgcn-amd-amdhsa-gfx908%xnack+.cui" +// CHECK-NOT: "-o" "hip-windows-filename-hip-amdgcn-amd-amdhsa-gfx908:xnack+.cui"