|
From: Larry D. <ldo...@re...> - 2015-07-28 03:14:00
|
Cary - On Mon, Jul 27, 2015 at 05:38:35AM +0000, Cary R. wrote: > The other issue is we also compile using clang so __GNUC__ does not likely work for all cases. I have not looked at this to check if there are issues, but I'm guessing we may need an #ifdef for MSVC just like we have for mingw. I agree in general, and of course this needs some thought and testing. It turns out clang does define __GNUC__, as shown here on Debian Jessie: $ clang --version Debian clang version 3.5.0-10 (tags/RELEASE_350/final) (based on LLVM 3.5.0) Target: x86_64-pc-linux-gnu Thread model: posix $ clang -dM -E - < /dev/null | grep GNU #define __GNUC_MINOR__ 2 #define __GNUC_PATCHLEVEL__ 1 #define __GNUC_STDC_INLINE__ 1 #define __GNUC__ 4 $ - Larry |