Re: [Boost-bugs] [Boost C++ Libraries] #9426: boost::atomic gcc-cas related specializations patch

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #9426: boost::atomic gcc-cas related specializations patch
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2013-11-23 18:12:48

#9426: boost::atomic gcc-cas related specializations patch
-------------------------------------+--------------------------
  Reporter:  Piotr Kobzda <pikob@…>  |      Owner:  timblechmann
      Type:  Patches                 |     Status:  new
 Milestone:  To Be Determined        |  Component:  atomic
   Version:  Boost 1.55.0            |   Severity:  Problem
Resolution:                          |   Keywords:
-------------------------------------+--------------------------

Comment (by Piotr Kobzda <pikob@…>):

 Just to make it clear, my patch was not ARM targeted (nor any other
 possible target that uses default atomics impl. currently) -- I just used
 ARM to verify gcc-cas implementation correctness.  I guess that current
 selection for ARMs is usually better than gcc-cas (at least for ARMs 6+).
 My primary intent was to allow use of gcc-cas on MIPSel platforms on which
 pthread mutexes are currently being selected by default.  BTW, I'd prefer
 to use lower level APIs for that, but currently I can not use nothing
 better then generic atomics for Mips implemented based on strong memory
 barrier only -- which most likely gives no noticeable advantages over
 implementation from gcc-cas.

 Regarding optional use of gcc-cas, if you think, that unconditional use of
 it (besides of earlier checks from platform.hpp and additional check for
 minimum gcc version supporting `__sync` API of course) is safe enough and
 better than default selection, I'd also opt for that.

-- 
Ticket URL: <https://svn.boost.org/trac/boost/ticket/9426#comment:6>
Boost C++ Libraries <http://www.boost.org/>
Boost provides free peer-reviewed portable C++ source libraries.