<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Mar 4, 2014 at 3:36 PM, Saleem Abdulrasool <span dir="ltr"><<a href="mailto:abdulras@fb.com" target="_blank">abdulras@fb.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class=""><br></div><div class="">
> ================<br>
> Comment at: include/llvm/ADT/Triple.h:334<br>
> @@ -329,2 +333,3 @@<br>
>   bool isOSCygMing() const {<br>
> -    return getOS() == Triple::Cygwin || getOS() == Triple::MinGW32;<br>
> +    return getOS() == Triple::Cygwin || getOS() == Triple::MinGW32 ||<br>
> +           (getOS() == Triple::Win32 && (getEnvironment() == Triple::Cygnus ||<br>
> ----------------<br>
> Are you planning to remove the MinGW32 OS enum?<br>
<br>
</div>Yes, I would like to rename Win32 to Windows and remove MinGW32.  However, Id like to do that in a subsequent pass.  I don’t have a strong opinion on that, and if you think that it is better done in one shot, Im happy to merge that into this change.</blockquote>
<div><br></div><div>No, I think it's better to do this incrementally, so that Clang and other consumers of LLVM can look for the new environment.</div></div></div></div>