[PATCH] D91605: [sanitizers] Implement GetTls on Solaris

Rainer Orth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 17 07:25:14 PST 2022


ro added a comment.

In D91605#3329375 <https://reviews.llvm.org/D91605#3329375>, @ro wrote:

> 



> As an experiment, I've tried to use `GetStaticTlsBoundary` instead.  It does indeed work on recent Solaris 11.4 and the resulting patch is at D120048 <https://reviews.llvm.org/D120048>.  I'm pretty certain that support for Solaris 11.3/Illumos which lack `dlpi_tls_modid` using `dlinfo(RTLD_SELF, RTLD_DI_LINKMAP)` can be added on top of that one, unbreaking the Illumos build.  This would avoid considerable duplication of the `dl_iterate_phdr` code, which is certainly a nice benefit.  I'll experiment with that route later.

Now posted as D120059 <https://reviews.llvm.org/D120059>.  It this approach is acceptable, the series of D119829 <https://reviews.llvm.org/D119829>, D120048 <https://reviews.llvm.org/D120048>, and D120059 <https://reviews.llvm.org/D120059> will supercede this one.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91605



More information about the cfe-commits mailing list