<div dir="ltr">No (just tested on windows with the cl driver and with the gcc style driver).  If you were to do this on Windows it wouldn't matter because that would be caught by the first branch #if defined(LLVM_ON_WIN32).  But you can run clang in Microsoft on Linux, for example.  And in that case neither LLVM_ON_WIN32 nor __GNUC__ would be defined.</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Aug 11, 2016 at 5:23 AM Michael Kruse 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">2016-08-11 14:21 GMT+02:00 Joerg Sonnenberger via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>>:<br>
> On Wed, Aug 10, 2016 at 10:17:22PM +0000, Zachary Turner wrote:<br>
>> __GNUC__ will be defined for GCC but __clang__ will not, right?<br>
><br>
> clang defines both, GCC only the former. So checking for the former<br>
> covers both.<br>
<br>
Does clang define __GNUC__ even in Microsoft mode?<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>