Commit 3c49c7d6 authored by Michael Lee's avatar Michael Lee Committed by Facebook Github Bot

Disable thread_local for all mobile targets

Summary: Android also has limited support for thread_local so we should not enable it at all as a precaution

Reviewed By: magedm

Differential Revision: D6884145

fbshipit-source-id: f39dd8fd5405ca0bc5be9b3538718a6f58058442
parent 5359a760
...@@ -30,9 +30,10 @@ ...@@ -30,9 +30,10 @@
#ifndef HAZPTR_TC #ifndef HAZPTR_TC
// If we are on an IOS device or simulator then do not use thread caching // If we are targeting a mobile device, do not use thread caching as a
// Performance is potentially suboptimal without thread local support // precaution. Performance is potentially suboptimal without thread
#if FOLLY_MOBILE && defined(__APPLE__) // local support
#if FOLLY_MOBILE
#define HAZPTR_TC false #define HAZPTR_TC false
#else #else
#define HAZPTR_TC true #define HAZPTR_TC true
...@@ -44,9 +45,10 @@ ...@@ -44,9 +45,10 @@
#endif #endif
#ifndef HAZPTR_PRIV #ifndef HAZPTR_PRIV
// If we are on an IOS device or simulator then do not use thread caching // If we are targeting a mobile device, do not use thread caching as a
// Performance is potentially suboptimal without thread local support // precaution. Performance is potentially suboptimal without thread
#if FOLLY_MOBILE && defined(__APPLE__) // local support
#if FOLLY_MOBILE
#define HAZPTR_PRIV false #define HAZPTR_PRIV false
#else #else
#define HAZPTR_PRIV true #define HAZPTR_PRIV true
......
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