[PATCH] D146351: sanitizer_common: Use plain thread_local for __sancov_lowest_stack definition.

[Vitaly Buka via Phabricator via llvm-commits]

vitalybuka accepted this revision.
vitalybuka added a comment.

I guess Android was that main reason was older API levels why don't use thread_local, probably our buildbot is still on them.
Wide use of thread_local may simplify sanitizers.

As I see thread_local is used in msan, which is not on Android, fuzzers, and asan fuchsia and netbsd.
However it's used in scudo, which is on Android

+Apple folks for any concerns.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D146351/new/

https://reviews.llvm.org/D146351