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