boost/detail/spinlock_sync.hpp incorrectly included for Intel C++ on Linux

[Anthony Williams]

On a linux-x86_64 system with gcc 4.1.3 and Intel 10.1, an include of boost/shared_ptr.hpp pulls in boost/detail/spinlock_sync.hpp, which fails to compile since the sync functions are undefined.

It appears this problem is due to the #if sequence in boost/detail/spinlock.hpp. The Intel compiler defines GNUC and GNUC_MINOR, but doesn't provide the intrinsics.

By adding "&& !defined(BOOST_INTEL)" to the appropriate #if line as in the attached patch, everything works.

[Anthony Williams]

Patch for boost/detail/spinlock.hpp

[Peter Dimov]

The same problem is pointed out in this thread:

​http://lists.boost.org/Archives/boost/2008/06/138322.php

but it seems that some installations of Intel C++ do handle the code fine:

​http://lists.boost.org/Archives/boost/2008/06/138337.php

The problem might be caused by the system headers:

​http://lists.boost.org/Archives/boost/2008/06/138341.php

It's not clear to me at the moment what the right fix is.

[Anthony Williams]

It appears to me to depend on the version of gcc used as the system compiler: certainly, if GNUC < 4, the problematic header won't be included. That's why I suggested the change I did. Of course, I'm not sure why Intel is defining GNUC anyway, but I guess it's for compatibility, like the Windows compiler defines _MSC_VER.

[notquitezeus@…]

I've observed the same problem on SuSE 10.3 x86_32 with ICC 10.1.15 (GCC primitives sync* not defined). I will give the patch a try once I get into the office tomorrow morning.

[Peter Dimov]

(In [48836]) Fix #2000.