Commit 896a7356 authored by Yedidya Feldblum's avatar Yedidya Feldblum Committed by Facebook GitHub Bot

cut unused macro FOLLY_TLS

Summary: Best to use C++ syntax `thread_local` instead since it is language-native and cross-platform.

Reviewed By: Orvid

Differential Revision: D27671626

fbshipit-source-id: 8ef0c487bfa20ab323067c05862998df25d2a53b
parent e1582a7f
......@@ -217,27 +217,6 @@ constexpr bool kIsSanitize = false;
#define FOLLY_GCC_DISABLE_NEW_SHADOW_WARNINGS /* empty */
#endif
/* Platform specific TLS support
* gcc implements __thread
* msvc implements __declspec(thread)
* the semantics are the same
* (but remember __thread has different semantics when using emutls (ex. apple))
*/
#if defined(_MSC_VER)
#define FOLLY_TLS __declspec(thread)
#elif defined(__GNUC__)
#define FOLLY_TLS __thread
#else
#error cannot define platform specific thread local storage
#endif
// disable FOLLY_TLS on 32 bit Apple/iOS
#if defined(__APPLE__) && FOLLY_MOBILE
#if (__SIZEOF_POINTER__ == 4)
#undef FOLLY_TLS
#endif
#endif
// It turns out that GNU libstdc++ and LLVM libc++ differ on how they implement
// the 'std' namespace; the latter uses inline namespaces. Wrap this decision
// up in a macro to make forward-declarations easier.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment