From: Jonathan Wakely Date: Wed, 12 Oct 2016 12:16:15 +0000 (+0100) Subject: Simplify std::call_once implementation X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=cfbdc34f25dac9e6568618f69e3ee46ff1714766;p=gcc.git Simplify std::call_once implementation * include/std/mutex [_GLIBCXX_HAVE_TLS] (_Once_call): Remove. (call_once) [_GLIBCXX_HAVE_TLS]: Simplify by removing _Once_call. From-SVN: r241034 --- diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index c02bbf03a9a..cda7761d6b6 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -1,5 +1,8 @@ 2016-10-12 Jonathan Wakely + * include/std/mutex [_GLIBCXX_HAVE_TLS] (_Once_call): Remove. + (call_once) [_GLIBCXX_HAVE_TLS]: Simplify by removing _Once_call. + * include/bits/stl_uninitialized.h (__uninitialized_default_novalue_n_1): Add missing return. * testsuite/20_util/specialized_algorithms/memory_management_tools/ diff --git a/libstdc++-v3/include/std/mutex b/libstdc++-v3/include/std/mutex index 4c6f0366f1c..e90006f43a0 100644 --- a/libstdc++-v3/include/std/mutex +++ b/libstdc++-v3/include/std/mutex @@ -579,21 +579,6 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION #ifdef _GLIBCXX_HAVE_TLS extern __thread void* __once_callable; extern __thread void (*__once_call)(); - - template::value>::__type> - struct _Once_call; - - template - struct _Once_call<_Tuple, _Index_tuple<_Ind...>> - { - static void - _S_call() - { - auto& __f_args = *static_cast<_Tuple*>(__once_callable); - std::__invoke(std::get<_Ind>(std::move(__f_args))...); - } - }; #else extern function __once_functor; @@ -613,17 +598,16 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION { // _GLIBCXX_RESOLVE_LIB_DEFECTS // 2442. call_once() shouldn't DECAY_COPY() -#ifdef _GLIBCXX_HAVE_TLS - auto __f_args = std::forward_as_tuple( - std::forward<_Callable>(__f), std::forward<_Args>(__args)...); - __once_callable = std::__addressof(__f_args); - __once_call = _Once_call::_S_call; -#else - unique_lock __functor_lock(__get_once_mutex()); - __once_functor = [&] { + auto __callable = [&] { std::__invoke(std::forward<_Callable>(__f), std::forward<_Args>(__args)...); }; +#ifdef _GLIBCXX_HAVE_TLS + __once_callable = std::__addressof(__callable); + __once_call = []{ (*(decltype(__callable)*)__once_callable)(); }; +#else + unique_lock __functor_lock(__get_once_mutex()); + __once_functor = __callable; __set_once_functor_lock_ptr(&__functor_lock); #endif