[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