<div dir="ltr">The 
<span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:12.8000001907349px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;float:none;display:inline!important;background-color:rgb(255,255,255)">__MSVCRT_VERSION__<span class=""> check seems like a nice improvement.</span></span><div><span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:12.8000001907349px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;float:none;display:inline!important;background-color:rgb(255,255,255)"><span class=""><br></span></span></div><div><span style="font-size:12.8000001907349px">If users are compiling binaries with MSVC, they generally know they have to either link the CRT statically or redistribute the appropriate CRT DLL. If I were a mingw64 user, I would not expect that binaries I build will have dependencies on things other than mingw64 itself and Windows unless I asked otherwise.</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">However, maybe it's more important for this to just work out of the box for developers than it is to worry about people building redistributed binaries. I just don't like the added build system complexity. =/</span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 30, 2015 at 2:13 PM, Mueller-Roemer, Johannes Sebastian <span dir="ltr"><<a href="mailto:Johannes.Sebastian.Mueller-Roemer@igd.fraunhofer.de" target="_blank">Johannes.Sebastian.Mueller-Roemer@igd.fraunhofer.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">But the problem with the dialog box etc. is the same on MinGW as on MSVC. And the MSVC executables won't run either if the appropriate runtimes are not installed on a target machine.<br>
<br>
Maybe as an compromise we could replace the "#ifdef _MSC_VER" now disabling it completely by an "#if defined(_MSC_VER) || __MSVCRT_VERSION__ >= 0x900". I could live with that. Although I'd still prefer a MinGW build to work automatically *and* have the no-message feature instead of silently disabling the feature.<br>
<br>
________________________________<br>
From: Reid Kleckner [<a href="mailto:rnk@google.com">rnk@google.com</a>]<br>
Sent: Friday, January 30, 2015 10:30 PM<br>
To: Mueller-Roemer, Johannes Sebastian<br>
Cc: <a href="mailto:reviews%2BD7224%2Bpublic%2B9c618652813cb116@reviews.llvm.org">reviews+D7224+public+9c618652813cb116@reviews.llvm.org</a>; <a href="mailto:bigcheesegs@gmail.com">bigcheesegs@gmail.com</a>; <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<span class="im HOEnZb">Subject: Re: [PATCH] Use a sufficiently current MSVCRT when using MinGW<br>
<br>
</span><div class="HOEnZb"><div class="h5">On Fri, Jan 30, 2015 at 1:17 PM, Mueller-Roemer, Johannes Sebastian <<a href="mailto:Johannes.Sebastian.Mueller-Roemer@igd.fraunhofer.de">Johannes.Sebastian.Mueller-Roemer@igd.fraunhofer.de</a><mailto:<a href="mailto:Johannes.Sebastian.Mueller-Roemer@igd.fraunhofer.de">Johannes.Sebastian.Mueller-Roemer@igd.fraunhofer.de</a>>> wrote:<br>
I assume you are referring to <a href="http://reviews.llvm.org/D7230" target="_blank">http://reviews.llvm.org/D7230</a>?<br>
<br>
Is it really desirable to disable the functionality for MinGW? Any recent MinGW is perfectly capable of compiling it with the right flags. My issue was only that running the generated executables requires the corresponding runtimes to be installed (also true for MSVC, but if that is installed the runtime is automatically installed as well)<br>
<br>
I don't like the idea of building mingw64 clang and LLVM executables that have surprising runtime dependencies on versions of the CRT that aren't shipped with either Windows or mingw64. It makes it very easy to build executables that won't run on other machines.<br>
</div></div></blockquote></div><br></div>