|
From: Cary R. <cy...@ya...> - 2015-07-31 22:47:44
|
Thanks for looking into this Larry. I was able to compile Icarus and run the test suite to verify that things still work correctly with clang. Cary On Monday, July 27, 2015 8:13 PM, Larry Doolittle <ldo...@re...> wrote: 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 |