Subject: Re: [Boost-bugs] [Boost C++ Libraries] #6308: Add sp_counted_base_aix.hpp using AIX atomic operations
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2011-12-23 17:28:15
#6308: Add sp_counted_base_aix.hpp using AIX atomic operations
-----------------------------------------+----------------------------------
Reporter: aaron.riekenberg@⦠| Owner: pdimov
Type: Feature Requests | Status: reopened
Milestone: To Be Determined | Component: smart_ptr
Version: Boost 1.48.0 | Severity: Optimization
Resolution: | Keywords:
-----------------------------------------+----------------------------------
Comment (by aaron.riekenberg@â¦):
I experimented with lwsync. It was slightly faster than sync in my tests
(around 300 nanoseconds). lwsync does not enforce ordering of stores
followed by loads
(http://www.ibm.com/developerworks/systems/articles/powerpc.html). So I
think I'll stick with sync for this reason, and to be consistent with
sp_counted_base_gcc_ppc.hpp.
I am attaching one last (hopefully!) patch which does the following:
1. Uses builtin functions sync() and isync() instead of inline asm.
2. Puts the atomic_* functions inside the boost::detail namespace - I
accidentally put them in the boost namespace previously.
Thanks again for all your help on this.
-- Ticket URL: <https://svn.boost.org/trac/boost/ticket/6308#comment:9> Boost C++ Libraries <http://www.boost.org/> Boost provides free peer-reviewed portable C++ source libraries.
This archive was generated by hypermail 2.1.7 : 2017-02-16 18:50:08 UTC