<div dir="ltr">This looks good to me.<div><br></div><div>The mingw CRT vs. MS CRT situation is confusing.  We have this code in llvm/Triple.h:</div><div><div>  /// \brief Is this a "Windows" OS targeting a "MSVCRT.dll" environment.</div>
<div>  bool isOSMSVCRT() const {</div><div>    return getOS() == Triple::Win32 || getOS() == Triple::MinGW32;</div><div>  }</div></div><div><br></div><div>The ifdef you're adding is the opposite of the above logic, but it makes sense because IIUC mingw has its own headers and small static libs to paper over the quirks of the MSVC CRT.  As far as libc++ is concerned, it's basically a different C runtime.</div>
<div><br></div><div>I believe the Triple code is mostly used when deciding how to lower things like large allocas to __chkstk.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 12, 2013 at 6:47 PM, G M <span dir="ltr"><<a href="mailto:gmisocpp@gmail.com" target="_blank">gmisocpp@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Here's another attempt at restoring the ability for libcxx to compile again on mingw 64. link errors should remain whatever problem they previously were or were not.</div>
<div> </div><div>See r187593 for more details regarding this patch's necessity.</div>
<div> </div><div>Please advise of suggestions or if you know it breaks anything that wasn't already broken and how (real error messages please) and I'll see if I can revise it accordingly. Otherwise I'm keen for it to be applied ASAP as the mingw build has been broken since the date r187593 was applied.</div>

<div> </div><div>This patch should also reduce the number of errors when compiling with Visual Studio though on VS2013 RC. libcxx doesn't yet compile (at least how I am building) but it is close. I will be submitting more patches to improve that situation a little shortly.</div>

<div> </div><div>Thanks</div></div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>