Re: [Iverilog-devel] MVSE 2015 hacks

[Cary R. <cy...@ya...>]

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