Boost C++ Libraries: Ticket #2575: Bug- Boost 1.36.0 on Itanium platform

component changed; owner set

Marshall Clow — Tue, 09 Dec 2008 17:31:01 GMT

owner set to John Maddock
component None → thread

John - can you take a look at this, since you've got the compiler?

The calls (and fns) are: boost/thread/win32/basic_timed_mutex.hpp: return !win32::interlocked_bit_test_and_set(&active_count,lock_flag_bit); boost/thread/win32/basic_timed_mutex.hpp: if(!win32::interlocked_bit_test_and_set(&active_count,lock_flag_bit)) boost/thread/win32/basic_timed_mutex.hpp: if(!win32::interlocked_bit_test_and_set(&active_count,event_set_flag_bit))

boost/thread/win32/thread_primitives.hpp: inline bool interlocked_bit_test_and_set(long* x,long bit) boost/thread/win32/thread_primitives.hpp: inline bool interlocked_bit_test_and_set(long* x,long bit) boost/thread/win32/thread_primitives.hpp: inline bool interlocked_bit_test_and_set(long* x,long bit)

owner changed

John Maddock — Tue, 09 Dec 2008 18:30:15 GMT

owner changed from John Maddock to Marshall Clow

I don't have access to the itanium compiler but looking at the docs and the headers:

_interlockedbittestandset is a compiler intrinsic *only* available on x86 and x86_64 architechtures, it should *not* be used on any other architechture.

I can't see a similar IA64 intrinsic (but could easily have missed it).

The function:

BOOLEAN WINAPI InterlockedBitTestAndSet(

in LONG volatile *Base, in LONG Bit

);

Appears to be universally available and offers the same functionality (in fact it just calls _interlockedbittestandset on x86 platforms).

HTH, John.

owner changed

Marshall Clow — Wed, 21 Jan 2009 17:19:27 GMT

owner changed from Marshall Clow to Anthony Williams

owner, status, milestone changed; cc set

viboes — Sun, 11 Dec 2011 02:40:49 GMT

cc viboes added
owner changed from Anthony Williams to viboes
status new → assigned
milestone Boost 1.38.0 → To Be Determined

viboes — Sun, 11 Dec 2011 15:11:13 GMT

After making the suggested replacement, I get this error when linking

msvc.link ..\..\..\bin.v2\libs\thread\test\test_thread.test\msvc-10.0\debug\threading-multi\test_thread.exe
test_thread.obj : error LNK2019: unresolved external symbol _InterlockedBitTestAndSet referenced in function "bool __cdecl boost::detail::win32::interlocked_bit_test_and_set(long *,long)" (?interlocked_bit_test_and_set@win32@detail@boost@@YA_NPAJJ@Z)
..\..\..\bin.v2\libs\thread\test\test_thread.test\msvc-10.0\debug\threading-multi\test_thread.exe : fatal error LNK1120: 1 unresolved externals

From which version is this primitive available?

keywords set

viboes — Sun, 11 Dec 2011 16:43:09 GMT

keywords interlocked added

type changed

viboes — Tue, 20 Dec 2011 18:07:31 GMT

type Bugs → Support Requests

Moved to support request until availability clarified.

attachment set

asonawane@… — Tue, 20 Dec 2011 18:52:39 GMT

attachment → thread_primitives.hpp

itanium patch

anonymous — Tue, 20 Dec 2011 18:52:54 GMT

I think it's been available on almost all Win32 versions since forever, a trivial program with just:

#include <windows.h>
int main()
{
   LONG i = 132;
   InterlockedBitTestAndSet(&i, 1);
}

compiles and links just fine for me on Vista with VC10.

Are you including windows.h or defining the prototype for InterlockedBitTestAndSet yourself?

anonymous — Tue, 20 Dec 2011 19:01:14 GMT

Replying to viboes:

Moved to support request until availability clarified.

Use the attached update (thread_primitives.hpp) replace and compile. Works with boost version 1.39.0 and MS-VS 2005. Later versions of boost not tested.

attachment set

viboes — Thu, 29 Dec 2011 16:20:46 GMT

attachment → 2575.patch

patch respect to trunk

type changed

viboes — Thu, 29 Dec 2011 16:22:45 GMT

type Support Requests → Patches

milestone changed

viboes — Sat, 07 Jan 2012 23:02:41 GMT

milestone To Be Determined → Boost 1.49.0

viboes — Tue, 17 Jan 2012 06:34:47 GMT

Committed in trunk at revision [76543].

status, milestone changed; resolution set

viboes — Mon, 28 May 2012 15:25:15 GMT

status assigned → closed
resolution → fixed
milestone Boost 1.49.0 → Boost 1.50.0

Committed in release branch at [78543]