dyung created this revision.
dyung added reviewers: ro, rnk, jkorous.
dyung added a project: clang.
Herald added subscribers: dexonsmith, fedor.sergeev.

This is a follow up to D63600 <https://reviews.llvm.org/D63600> that attempts 
to fix the test to work on all platforms including Windows (which the previous 
patch broke) and Solaris (which is the original motivation for this patch).


https://reviews.llvm.org/D63678

Files:
  clang/test/Driver/cl-response-file.c


Index: clang/test/Driver/cl-response-file.c
===================================================================
--- clang/test/Driver/cl-response-file.c
+++ clang/test/Driver/cl-response-file.c
@@ -4,7 +4,7 @@
 
 
 
-// RUN: echo '/I%S\Inputs\cl-response-file\ /DFOO=2' > %t.rsp
+// RUN: printf '%%s\n' '/I%S\Inputs\cl-response-file\ /DFOO=2' > %t.rsp
 // RUN: %clang_cl /c -### @%t.rsp -- %s 2>&1 | FileCheck %s
 
 // CHECK: "-I" "{{.*}}\\Inputs\\cl-response-file\\" "-D" "FOO=2"


Index: clang/test/Driver/cl-response-file.c
===================================================================
--- clang/test/Driver/cl-response-file.c
+++ clang/test/Driver/cl-response-file.c
@@ -4,7 +4,7 @@
 
 
 
-// RUN: echo '/I%S\Inputs\cl-response-file\ /DFOO=2' > %t.rsp
+// RUN: printf '%%s\n' '/I%S\Inputs\cl-response-file\ /DFOO=2' > %t.rsp
 // RUN: %clang_cl /c -### @%t.rsp -- %s 2>&1 | FileCheck %s
 
 // CHECK: "-I" "{{.*}}\\Inputs\\cl-response-file\\" "-D" "FOO=2"
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to