[PATCH] D31947: [scudo] Android support groundwork

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 17 13:23:41 PDT 2017


cryptoad added a comment.

In https://reviews.llvm.org/D31947#728499, @srhines wrote:

> In https://reviews.llvm.org/D31947#728470, @kcc wrote:
>
> > >> Android doesn't support thread_local
> >
> > eugenis@, Is that still true? 
> >  Anyway, would you like to look at this from the Android POV?
>
>
> That isn't true. Android does support thread_local. This patch should not have a different Android path.


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.


https://reviews.llvm.org/D31947





More information about the llvm-commits mailing list