[PATCH] D31947: [scudo] Android support groundwork

[Renato Golin via Phabricator via llvm-commits]

rengolin added a subscriber: jasonk.
rengolin added a comment.

In https://reviews.llvm.org/D31947#728503, @cryptoad wrote:

> To be more specific, thread_local is supported via emutls (llvm/projects/compiler-rt/lib/builtins/emutls.c), which itself calls malloc to allocate the TLS.
>  Which unfortunately doesn't work well for an allocator that replaces malloc.


I thought we had fixed this one by asking the Android devs to implement proper TLS on Android. They said it was simple and wasn't there because no one needed it.

@srhines, were you at the meeting with @jasonk, @kcc and some Android devs at Google, maybe 2 years ago? We discussed how `emutls` could be completely avoided.

Or maybe I remember it wrong...

--renato


https://reviews.llvm.org/D31947