Boost C++ Libraries: Ticket #2000: boost/detail/spinlock_sync.hpp incorrectly included for Intel C++ on Linux

attachment set

Anthony Williams — Mon, 09 Jun 2008 14:21:38 GMT

attachment → spinlock.patch

Patch for boost/detail/spinlock.hpp

status changed

Peter Dimov — Tue, 10 Jun 2008 09:13:41 GMT

status new → assigned

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 — Wed, 11 Jun 2008 21:31:50 GMT

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.

milestone changed

Peter Dimov — Thu, 21 Aug 2008 15:08:12 GMT

milestone Boost 1.36.0 → Boost 1.37.0

notquitezeus@… — Thu, 11 Sep 2008 03:51:37 GMT

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.

status changed; resolution set

Peter Dimov — Wed, 17 Sep 2008 22:43:16 GMT

status assigned → closed
resolution → fixed

(In [48836]) Fix #2000.