<div dir="ltr">Ahh, yea I thought I was using LLVM_ON_WIN32 instead of _MSC_VER, but it turns out I'm using _MSC_VER.  TBH I will defer to you guys, I'm not an expert on all the different platforms and their configurations / definitions.  If having __clang__ is at worst redundant, seems like it wouldn't hurt to leave it and be on the safe side, but if someone out there is absolutely sure it's not necessary I can remove it.</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Aug 11, 2016 at 12:03 PM Joerg Sonnenberger via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Aug 11, 2016 at 06:35:42PM +0000, Zachary Turner wrote:<br>
> No (just tested on windows with the cl driver and with the gcc style<br>
> driver).  If you were to do this on Windows it wouldn't matter because that<br>
> would be caught by the first branch #if defined(LLVM_ON_WIN32).  But you<br>
> can run clang in Microsoft on Linux, for example.  And in that case neither<br>
> LLVM_ON_WIN32 nor __GNUC__ would be defined.<br>
<br>
But don't we hit the _MSC_VER case in that case?<br>
<br>
Joerg<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>