Boost C++ Libraries: Ticket #2525: More architectures that do not support __sync_fetch_and_add_4

attachment set

Sat, 22 Nov 2008 11:31:09 GMT

attachment → atomic_count.patch

attachment set

Sat, 22 Nov 2008 11:31:38 GMT

attachment → atomic_count_gcc.patch

attachment set

smr@… — Wed, 17 Dec 2008 05:28:49 GMT

attachment → sp_counted_base.patch

smr@… — Wed, 17 Dec 2008 05:30:23 GMT

Added sp_counted_base.patch to address another manifestation in sp_counted_base.hpp.

status, milestone changed

Peter Dimov — Sat, 28 Feb 2009 17:05:35 GMT

status new → assigned
milestone Boost 1.38.0 → Boost 1.39.0

Peter Dimov — Sun, 01 Mar 2009 18:01:21 GMT

(In [51517]) Refs #2525 (fixed in trunk.)

status changed; resolution set

Peter Dimov — Mon, 02 Mar 2009 16:42:30 GMT

status assigned → closed
resolution → fixed

(In [51538]) Merge [51517] to release. Closes #2525.

status changed; resolution deleted

Peter Dimov — Tue, 03 Mar 2009 15:29:27 GMT

status closed → reopened
resolution fixed

Reopening due to the claim in

http://lists.boost.org/Archives/boost/2007/11/130479.php

that g++ 4.2 has __sync operations on __sparc__. Steven, can you please confirm or deny it?

smr@… — Mon, 09 Mar 2009 03:15:27 GMT

On the sparc to which I have access (a sparc64 running Debian Linux), I can attest that g++ 4.2 does *not* have all the __sync primitives used in Boost. I created a test program using all the primitives I could find with "rgrep __sync_ *" in the Boost root source directory (SVN head).

Here is the transcript.

(sid)smr@smetana:~$ uname -a
Linux smetana 2.6.26-1-sparc64-smp #1 SMP Sat Jan 10 18:53:17 UTC 2009 sparc64 GNU/Linux
(sid)smr@smetana:~$ cat sync-primitives.cc
/* Test that uses all the __sync primitives found
   by grepping the boost sources.
*/
int main( int ac, char* av[] )
{
    long value = 0;
    __sync_lock_release( &value );
    return __sync_fetch_and_add( &value, 1 )
        +  __sync_val_compare_and_swap( &value, 1, 2 )
        +  __sync_add_and_fetch( &value, 1 )
        +  __sync_lock_test_and_set( &value, 1 );
}
(sid)smr@smetana:~$ g++-4.1 sync-primitives.cc
/tmp/ccGOFRyI.o: In function `main':
sync-primitives.cc:(.text+0x24): undefined reference to `__sync_fetch_and_add_4'
sync-primitives.cc:(.text+0x44): undefined reference to `__sync_val_compare_and_swap_4'
sync-primitives.cc:(.text+0x60): undefined reference to `__sync_add_and_fetch_4'
sync-primitives.cc:(.text+0x7c): undefined reference to `__sync_lock_test_and_set_4'
collect2: ld returned 1 exit status
(sid)smr@smetana:~$ g++-4.2 sync-primitives.cc
/tmp/ccQg4NKe.o: In function `main':
sync-primitives.cc:(.text+0x28): undefined reference to `__sync_fetch_and_add_4'
sync-primitives.cc:(.text+0x48): undefined reference to `__sync_val_compare_and_swap_4'
sync-primitives.cc:(.text+0x64): undefined reference to `__sync_add_and_fetch_4'
collect2: ld returned 1 exit status
(sid)smr@smetana:~$ g++-4.3 sync-primitives.cc
/tmp/cc8anhqk.o: In function `main':
sync-primitives.cc:(.text+0x28): undefined reference to `__sync_fetch_and_add_4'
sync-primitives.cc:(.text+0x48): undefined reference to `__sync_val_compare_and_swap_4'
sync-primitives.cc:(.text+0x64): undefined reference to `__sync_add_and_fetch_4'
collect2: ld returned 1 exit status

status changed; resolution set

Peter Dimov — Mon, 09 Mar 2009 15:25:33 GMT

status reopened → closed
resolution → fixed

OK, closing as fixed, thanks.