[PATCH] D37530: [MinGW] Allow overriding which version of msvcrt to link to

2017-09-06 Thread Martin Storsjö via Phabricator via cfe-commits
mstorsjo added a comment. In https://reviews.llvm.org/D37530#862644, @rnk wrote: > What do you think of letting people spell this as `-lmsvcrt120`? We could > forward those options and suppress our implicit addition of `-lmsvcrt` if we > see `-lmsvcr*` anywhere. That might work and would prob

[PATCH] D37530: [MinGW] Allow overriding which version of msvcrt to link to

2017-09-06 Thread Reid Kleckner via Phabricator via cfe-commits
rnk added a comment. What do you think of letting people spell this as `-lmsvcrt120`? We could forward those options and suppress our implicit addition of `-lmsvcrt` if we see `-lmsvcr*` anywhere. Comment at: lib/Driver/ToolChains/MinGW.cpp:161 if (TC.getArch() == llvm

[PATCH] D37530: [MinGW] Allow overriding which version of msvcrt to link to

2017-09-06 Thread Martin Storsjö via Phabricator via cfe-commits
mstorsjo created this revision. This allows linking to e.g. msvcr120.dll or ucrtbase.dll instead of the unversioned (and officially unsupported) msvcrt.dll. In GCC setups, this can be overridden by using custom spec files, but this isn't supported in clang. This is just an initial attempt at im