- User Since
- Dec 10 2015, 8:55 PM (200 w, 5 d)
Jan 16 2017
Addresses comments from @mclow.lists.
Addresses comments from @EricWF.
Jan 11 2017
Adresses comments from @kubabrecka: minor changes on function names. Rename __libcpp_atomic_* to __libcpp_atomic_refcount_*.
Jan 10 2017
Addressed previous issues in the comments. The patch still shows consistent perf uplift in proprietary benchmark on shared_ptr.
Jan 3 2017
Minor fix, remove redundant inlines.
Move the header back in its place, and only copy over necessary parts. Now call __atomic_add_fetch from inside the functions.
Nov 15 2016
Nov 7 2016
Oct 31 2016
Looks good to me.
Notice that the performance gain can only be observed when compiled with the updated C++ header files.
Oct 12 2016
Thanks for pointing out. It's true that it may cause ABI breakage. It would be nice to keep compatibility while getting the performance benefits from inlining.
Oct 3 2016
Addresses comments from @halyavin, rename "atomic_support.h" to "__atomic_support" to avoid collisions with application headers.
Sep 27 2016
Ping agian. Thanks in advance for the reviews.
Sep 19 2016
Ping. Are there any updates on this? Thanks very much.
Sep 12 2016
Address comments from @arsenm.
Sep 9 2016
Ping. Are there any updates or thoughts on this patch? Thanks.