Author: ericwf Date: Tue Jul 11 18:16:33 2017 New Revision: 307746 URL: http://llvm.org/viewvc/llvm-project?rev=307746&view=rev Log: Revert "[libc++] Refactoring __sync_* builtins; NFC (Reland)"
This reverts commit r307595. The commit had some issues that needed to first be addressed in review. Removed: libcxx/trunk/include/__atomic_support Modified: libcxx/trunk/include/__refstring libcxx/trunk/src/locale.cpp libcxx/trunk/src/support/runtime/exception_fallback.ipp libcxx/trunk/src/support/runtime/new_handler_fallback.ipp Removed: libcxx/trunk/include/__atomic_support URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/include/__atomic_support?rev=307745&view=auto ============================================================================== --- libcxx/trunk/include/__atomic_support (original) +++ libcxx/trunk/include/__atomic_support (removed) @@ -1,31 +0,0 @@ -// -*- C++ -*- -//===------------------------ __atomic_support -----------------------------===// -// -// The LLVM Compiler Infrastructure -// -// This file is dual licensed under the MIT and the University of Illinois Open -// Source Licenses. See LICENSE.TXT for details. -// -//===----------------------------------------------------------------------===// - -#ifndef _LIBCPP___ATOMIC_SUPPORT -#define _LIBCPP___ATOMIC_SUPPORT - -#include <__config> - -template<typename T> -T __libcpp_sync_add_and_fetch(T *ptr, T value) { - return __sync_add_and_fetch(ptr, value); -} - -template<typename T> -T __libcpp_sync_fetch_and_add(T *ptr, T value) { - return __sync_fetch_and_add(ptr, value); -} - -template<typename T> -T __libcpp_sync_lock_test_and_set(T *ptr, T value) { - return __sync_lock_test_and_set(ptr, value); -} - -#endif //_LIBCPP___ATOMIC_SUPPORT Modified: libcxx/trunk/include/__refstring URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/include/__refstring?rev=307746&r1=307745&r2=307746&view=diff ============================================================================== --- libcxx/trunk/include/__refstring (original) +++ libcxx/trunk/include/__refstring Tue Jul 11 18:16:33 2017 @@ -14,7 +14,6 @@ #include <stdexcept> #include <cstddef> #include <cstring> -#include <__atomic_support> #ifdef __APPLE__ #include <dlfcn.h> #include <mach-o/dyld.h> @@ -84,7 +83,7 @@ __libcpp_refstring::__libcpp_refstring(c : __imp_(s.__imp_) { if (__uses_refcount()) - __libcpp_sync_add_and_fetch(&rep_from_data(__imp_)->count, 1); + __sync_add_and_fetch(&rep_from_data(__imp_)->count, 1); } inline @@ -93,10 +92,10 @@ __libcpp_refstring& __libcpp_refstring:: struct _Rep_base *old_rep = rep_from_data(__imp_); __imp_ = s.__imp_; if (__uses_refcount()) - __libcpp_sync_add_and_fetch(&rep_from_data(__imp_)->count, 1); + __sync_add_and_fetch(&rep_from_data(__imp_)->count, 1); if (adjust_old_count) { - if (__libcpp_sync_add_and_fetch(&old_rep->count, count_t(-1)) < 0) + if (__sync_add_and_fetch(&old_rep->count, count_t(-1)) < 0) { ::operator delete(old_rep); } @@ -108,7 +107,7 @@ inline __libcpp_refstring::~__libcpp_refstring() { if (__uses_refcount()) { _Rep_base* rep = rep_from_data(__imp_); - if (__libcpp_sync_add_and_fetch(&rep->count, count_t(-1)) < 0) { + if (__sync_add_and_fetch(&rep->count, count_t(-1)) < 0) { ::operator delete(rep); } } Modified: libcxx/trunk/src/locale.cpp URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/src/locale.cpp?rev=307746&r1=307745&r2=307746&view=diff ============================================================================== --- libcxx/trunk/src/locale.cpp (original) +++ libcxx/trunk/src/locale.cpp Tue Jul 11 18:16:33 2017 @@ -28,7 +28,6 @@ #define _CTYPE_DISABLE_MACROS #endif #include "cwctype" -#include "__atomic_support" #include "__sso_allocator" #if defined(_LIBCPP_MSVCRT) || defined(__MINGW32__) #include "support/win32/locale_win32.h" @@ -668,7 +667,7 @@ locale::id::__get() void locale::id::__init() { - __id_ = __libcpp_sync_add_and_fetch(&__next_id, 1); + __id_ = __sync_add_and_fetch(&__next_id, 1); } // template <> class collate_byname<char> Modified: libcxx/trunk/src/support/runtime/exception_fallback.ipp URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/src/support/runtime/exception_fallback.ipp?rev=307746&r1=307745&r2=307746&view=diff ============================================================================== --- libcxx/trunk/src/support/runtime/exception_fallback.ipp (original) +++ libcxx/trunk/src/support/runtime/exception_fallback.ipp Tue Jul 11 18:16:33 2017 @@ -9,7 +9,6 @@ //===----------------------------------------------------------------------===// #include <cstdio> -#include <__atomic_support> namespace std { @@ -21,13 +20,13 @@ _LIBCPP_SAFE_STATIC static std::unexpect unexpected_handler set_unexpected(unexpected_handler func) _NOEXCEPT { - return __libcpp_sync_lock_test_and_set(&__unexpected_handler, func); + return __sync_lock_test_and_set(&__unexpected_handler, func); } unexpected_handler get_unexpected() _NOEXCEPT { - return __libcpp_sync_fetch_and_add(&__unexpected_handler, (unexpected_handler)0); + return __sync_fetch_and_add(&__unexpected_handler, (unexpected_handler)0); } @@ -42,13 +41,14 @@ void unexpected() terminate_handler set_terminate(terminate_handler func) _NOEXCEPT { - return __libcpp_sync_lock_test_and_set(&__terminate_handler, func); + return __sync_lock_test_and_set(&__terminate_handler, func); } terminate_handler get_terminate() _NOEXCEPT { - return __libcpp_sync_fetch_and_add(&__terminate_handler, (terminate_handler)0); + return __sync_fetch_and_add(&__terminate_handler, (terminate_handler)0); + } #ifndef __EMSCRIPTEN__ // We provide this in JS Modified: libcxx/trunk/src/support/runtime/new_handler_fallback.ipp URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/src/support/runtime/new_handler_fallback.ipp?rev=307746&r1=307745&r2=307746&view=diff ============================================================================== --- libcxx/trunk/src/support/runtime/new_handler_fallback.ipp (original) +++ libcxx/trunk/src/support/runtime/new_handler_fallback.ipp Tue Jul 11 18:16:33 2017 @@ -8,8 +8,6 @@ // //===----------------------------------------------------------------------===// -#include <__atomic_support> - namespace std { _LIBCPP_SAFE_STATIC static std::new_handler __new_handler; @@ -17,13 +15,13 @@ _LIBCPP_SAFE_STATIC static std::new_hand new_handler set_new_handler(new_handler handler) _NOEXCEPT { - return __libcpp_sync_lock_test_and_set(&__new_handler, handler); + return __sync_lock_test_and_set(&__new_handler, handler); } new_handler get_new_handler() _NOEXCEPT { - return __libcpp_sync_fetch_and_add<new_handler>(&__new_handler, nullptr); + return __sync_fetch_and_add(&__new_handler, nullptr); } } // namespace std _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits