Re: r268797 - Fix sysroot-prefix.c on Windows (/ vs \).

2016-05-06 Thread Nico Weber via cfe-commits
On Fri, May 6, 2016 at 5:43 PM, Yaron Keren via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Hi Nico, > > In other tests, {{/|}} or {{/|\\}} are used match the forward or > backward slash(es). clang escapes \ so we get two of them in output and > four in the LIT test. > > It would be nic

Re: r268797 - Fix sysroot-prefix.c on Windows (/ vs \).

2016-05-06 Thread Yaron Keren via cfe-commits
Hi Nico, In other tests, {{/|}} or {{/|\\}} are used match the forward or backward slash(es). clang escapes \ so we get two of them in output and four in the LIT test. It would be nice to have more elegant construct for this general problem, to avoid tests like: "{{.*}}/Inputs/mingw_mingw_or

r268797 - Fix sysroot-prefix.c on Windows (/ vs \).

2016-05-06 Thread Nico Weber via cfe-commits
Author: nico Date: Fri May 6 16:17:32 2016 New Revision: 268797 URL: http://llvm.org/viewvc/llvm-project?rev=268797&view=rev Log: Fix sysroot-prefix.c on Windows (/ vs \). Modified: cfe/trunk/test/Preprocessor/sysroot-prefix.c Modified: cfe/trunk/test/Preprocessor/sysroot-prefix.c URL: htt