[PATCH] D31947: [scudo] Android support groundwork

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 18 08:00:40 PDT 2017


cryptoad added a comment.

In https://reviews.llvm.org/D31947#728956, @dvyukov wrote:

> We have a (real) TLS slot for tsan (TLS_SLOT_TSAN). I wonder if we can re-use it for other sanitizers and scudo. Sanitizers are mutually exclusive, so there should be no problems using TLS_SLOT_TSAN is asan/msan. Scudo is not supposed to be used with sanitizers, right? So it seems we can just (ab)use TLS_SLOT_TSAN as well. Accessing it is just few machine instructions (see TLS_SLOT_TSAN in tsan_platform_linux.cc).


Scudo indeed is mutually exclusive with the other Sanitizers, so this option sounds like it could work!


https://reviews.llvm.org/D31947





More information about the llvm-commits mailing list