[PATCH] D22426: Fix automatic detection of ARM MSVC toolset in clang.exe

Reid Kleckner via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 18 11:12:01 PDT 2016


rnk added a comment.

In https://reviews.llvm.org/D22426#486129, @DaveBartolomeo wrote:

> One reasonable solution would be to choose the toolset that is hosted on the same architecture as the host of clang.exe (e.g. x64-hosted Clang looks for x64-hosted MSVC). If this sounds good, I can make that change in a follow-up commit.


I like that, it seems more principled. In the meantime, I think we should go with VC/amd64_{x86|arm}/bin instead. Maybe I'm biased, but I've seem more linker failures due to address space exhaustion than 32-bit Windows kernels.


https://reviews.llvm.org/D22426





More information about the cfe-commits mailing list