Re: r263344 - clang-cl: Add a test for the interaction of /Yc and /showIncludes.

2016-03-13 Thread Nico Weber via cfe-commits
r263400 probably helps, sorry about the breakage. As far as I can tell, ~all breakages of this bot are fixed by adding a `// REQUIRES: x86-registered-target` to a new test, so that's something you could add yourself to get your bots back green until whoever broke it comes back online to take a loo

Re: r263344 - clang-cl: Add a test for the interaction of /Yc and /showIncludes.

2016-03-13 Thread Renato Golin via cfe-commits
On 13 March 2016 at 02:56, Nico Weber via cfe-commits wrote: > Author: nico > Date: Sat Mar 12 13:55:59 2016 > New Revision: 263344 > > URL: http://llvm.org/viewvc/llvm-project?rev=263344&view=rev > Log: > clang-cl: Add a test for the interaction of /Yc and /showIncludes. > > We almost get this ri

r263344 - clang-cl: Add a test for the interaction of /Yc and /showIncludes.

2016-03-12 Thread Nico Weber via cfe-commits
Author: nico Date: Sat Mar 12 13:55:59 2016 New Revision: 263344 URL: http://llvm.org/viewvc/llvm-project?rev=263344&view=rev Log: clang-cl: Add a test for the interaction of /Yc and /showIncludes. We almost get this right, but not completely (see FIXME). It looks like /FI headers generally aren