__GNUC__ isn't always defined

[Naville]

We're cross-compiling Z3 for Windows using MSVC Sysroot and clang-cl on Linux.
The compile steps are fine, but linking would fail due to _tzcnt_u64 symbol not defined.

I'm clueless with how MSVC works, so here are some educated assumptions:
From https://github.com/llvm/llvm-project/blob/07d2709a17860a202d91781769a88837e4fb5f2a/clang/lib/Frontend/InitPreprocessor.cpp#L868 we could see that while __clang__ and __llvm__ are always defined, but __GNUC__ is only defined when building with GNU C Version specified, which is not the case when using Clang-CL.

This results in the following Z3 code:

z3/src/util/mpz.cpp

Line 50 in 0c16d34

#if defined(__GNUC__)

and

z3/src/util/mpz.cpp

Line 65 in 0c16d34

#if defined(__GNUC__)

falls back to using #define _trailing_zeros**(X) _tzcnt_u**(X)

Grepping through Windows SDK:

WinSDK/VC/Tools/MSVC/14.26.28801/include/immintrin.h:extern unsigned int     _tzcnt_u32(unsigned int);
WinSDK/VC/Tools/MSVC/14.26.28801/include/ammintrin.h:unsigned int _tzcnt_u32(unsigned int);
WinSDK/VC/Tools/MSVC/14.26.28801/bin/Hostx64/x86/c1xx.dll: binary file matches
WinSDK/VC/Tools/MSVC/14.26.28801/bin/Hostx64/x86/c1.dll: binary file matches

We can see that these two builtins are listed as an external function, which makes compilation success.
However without c1xx and c1.dll to lower those intrinsics calls, clang generates them to actual lib call, which fails in linking due to those functions only exists as intrinsics

[Naville]

I marked those two checks to check for __clang__ too and it worked. which confirmed my guesses

[Naville]

For this specific case, another even better alternative would be using __has_builtin

#if defined __has_builtin
#  if __has_builtin (__builtin_ctz)
#    define _trailing_zeros32(X) __builtin_ctz(X)
# endif
#  if __has_builtin (__builtin_ctzll)
#    define _trailing_zeros64(X) __builtin_ctzll(X)
#  endif
#endif

[Naville]

ping