Repository: mesos Updated Branches: refs/heads/master 4f4aff955 -> 310c0d62d
Removed workaround for lack of `thread_local` on OS X. Until XCode 8, Apple's build toolchain did not support the `thread_local` feature from C++11. Hence, Stout provided a `THREAD_LOCAL` macro, which expanded to `__thread` on OS X and `thread_local` on other platforms. XCode 8 added support for `thread_local`, so we can remove this workaround. Note that XCode 8 requires OS X 10.11.5 or newer. Review: https://reviews.apache.org/r/58310 Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/7e8b4e42 Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/7e8b4e42 Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/7e8b4e42 Branch: refs/heads/master Commit: 7e8b4e4286d336063845949bd0148d03481d754c Parents: 4f4aff9 Author: Neil Conway <[email protected]> Authored: Mon Apr 10 09:32:03 2017 -0700 Committer: Neil Conway <[email protected]> Committed: Wed May 3 14:19:26 2017 -0700 ---------------------------------------------------------------------- 3rdparty/stout/include/stout/thread_local.hpp | 11 ----------- 1 file changed, 11 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/7e8b4e42/3rdparty/stout/include/stout/thread_local.hpp ---------------------------------------------------------------------- diff --git a/3rdparty/stout/include/stout/thread_local.hpp b/3rdparty/stout/include/stout/thread_local.hpp index 7ffc809..dacf212 100644 --- a/3rdparty/stout/include/stout/thread_local.hpp +++ b/3rdparty/stout/include/stout/thread_local.hpp @@ -13,17 +13,6 @@ #ifndef __STOUT_THREAD_LOCAL_HPP__ #define __STOUT_THREAD_LOCAL_HPP__ -// A wrapper around the thread local storage attribute. The default -// clang on OSX does not support the c++11 standard `thread_local` -// intentionally until a higher performance implementation is -// released. See https://devforums.apple.com/message/1079348#1079348 -// Until then, we use `__thread` on OSX instead. -// We required that THREAD_LOCAL is only used with POD types as this -// is a requirement of `__thread`. -#ifdef __APPLE__ -#define THREAD_LOCAL __thread -#else #define THREAD_LOCAL thread_local -#endif #endif // __STOUT_THREAD_LOCAL_HPP__
