dim created this revision. dim added reviewers: EricWF, mclow.lists. dim added subscribers: majnemer, jroelofs, cfe-commits. Herald added a subscriber: aemerson.
In rL241532, atomic_support.h was added, which provides handling of atomic operations for libc++. When atomic builtins are not available, it emits a warning about being unsupported, but it still provides a number of stubs for the required functions. However, it misses a stub for __libcpp_relaxed_store(). Add it, by using the same implementation as for __libcpp_atomic_store(). (Note that I encountered this on arm-freebsd, which still defaults to armv4, and does not have the runtime libcalls to support atomic builtins. For now, I have simply disabled using them.) http://reviews.llvm.org/D13051 Files: src/include/atomic_support.h Index: src/include/atomic_support.h =================================================================== --- src/include/atomic_support.h +++ src/include/atomic_support.h @@ -103,6 +103,13 @@ *__dest = __val; } +template <class _ValueType, class _FromType> +inline _LIBCPP_INLINE_VISIBILITY +void __libcpp_relaxed_store(_ValueType* __dest, _FromType __val) +{ + *__dest = __val; +} + template <class _ValueType> inline _LIBCPP_INLINE_VISIBILITY _ValueType __libcpp_atomic_load(_ValueType const* __val,
Index: src/include/atomic_support.h =================================================================== --- src/include/atomic_support.h +++ src/include/atomic_support.h @@ -103,6 +103,13 @@ *__dest = __val; } +template <class _ValueType, class _FromType> +inline _LIBCPP_INLINE_VISIBILITY +void __libcpp_relaxed_store(_ValueType* __dest, _FromType __val) +{ + *__dest = __val; +} + template <class _ValueType> inline _LIBCPP_INLINE_VISIBILITY _ValueType __libcpp_atomic_load(_ValueType const* __val,
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits