r363985 - [test][Driver] Fix Clang :: Driver/cl-response-file.c
Rainer Orth via cfe-commits
cfe-commits at lists.llvm.org
Fri Jun 21 14:46:01 PDT 2019
Hi Douglas,
> I figured it out. Because you are using printf, you no longer need the
> '\\c'. If I use the following RUN line, the test passes on both
> Windows/linux (I don't have solaris to test unfortunately):
>
> // RUN: printf '%%s\n' '/I%S\Inputs\cl-response-file\ /DFOO=2' > %t.rsp
I've now tested it myself successfully on amd64-pc-solaris2.11. I'd
figured the \\c -> \c part out already, but had to leave before
understanding the %s problem.
> I can submit this patch for you if you would like.
That would be great, thanks a lot.
Rainer
More information about the cfe-commits
mailing list