<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Mar 29, 2013 at 11:57 AM, Timur Iskhodzhanov <span dir="ltr"><<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">2013/3/28 Anton Korobeynikov <<a href="mailto:asl@math.spbu.ru">asl@math.spbu.ru</a>>:<br>
<div class="im">>> How can having an MSVC compatible compiler be to the detriment of clang and<br>
>> llvm? No one is trying to break mingw here, merely add support for something<br>
> Just to make stuff clear: I just wanted proper naming which will be<br>
> non-confusing. Right now we have:<br>
>  - isTargetWindows() which really means "msvc-compabile"<br>
>  - isTargetWin32() which means "everything on windows", so Windows +<br>
> Mingw + Cygwin<br>
</div>Minor correction: currently isTargetWin32 means 32-bits (not<br>
"everything"), Windows + Mingw, not Cygwin.<br>
So this is actually even more confusing...<br>
<div class="im"><br>
>  - isTargetWin64() is is basically 64-bit version of isTargetWin32(),<br>
> but strictly speaking is slightly different<br>
><br>
> This naming while being the historical artifact is extremely<br>
> confusing. For me it seems the best solution will be something like<br>
> this:<br>
>  - isTargetMingw() - with obvious meaning<br>
>  - isTargetMSVC() - with obvious meaning<br>
>  - isTargetWindows() which will include all the flavours (so only OS<br>
> will matter here)<br>
>  - isTargetWindows() can be combined with existing 32/64 bit checks<br>
> This way we'll end with something being non-ambiguos.<br>
</div>I think this is a much better naming than we have now, I'll prepare a<br>
patch for that.<br>
<br>
Should we change the ***-pc-win32 triple to ***-pc-msvc ?<br>
<div class="im HOEnZb"><br></div></blockquote><div><br></div><div style>I agree that explicitness is good here, and win32 *is* confusing when it doesn't specify which compiler is meant. Of course the internal isTarget*() APIs should be changed to match the chosen convention as well.</div>
<div style><br></div><div style>Eli</div><div style><br></div></div><br></div></div>