Re: r297861 - Don't use MSVC Setup Api on MinGW.

2017-03-16 Thread Zachary Turner via cfe-commits
Pretty sure It's the latter. I don't use MinGW enough enough to know how compatible it is with msvc, but this uses a bunch of COM APIs and funny Microsoftisms like declspec(novtable), declspec(uuid), etc, and the MinGW bot was throwing hundreds of compiler errors. If there's a way to make it work I

Re: r297861 - Don't use MSVC Setup Api on MinGW.

2017-03-16 Thread Nico Weber via cfe-commits
Should this be a compile-time thing? I thought in general we try to make clang-built-with-msvc and clang-built-with-mingw behave roughly the same except for a different default triple, i.e. maybe this should be keyed off the triple at runtime instead of with a compile-time ifdef. Or do some of the

r297861 - Don't use MSVC Setup Api on MinGW.

2017-03-15 Thread Zachary Turner via cfe-commits
Author: zturner Date: Wed Mar 15 12:09:36 2017 New Revision: 297861 URL: http://llvm.org/viewvc/llvm-project?rev=297861&view=rev Log: Don't use MSVC Setup Api on MinGW. Modified: cfe/trunk/lib/Driver/ToolChains/MSVC.cpp Modified: cfe/trunk/lib/Driver/ToolChains/MSVC.cpp URL: http://llvm.org