[PATCH] D91605: [sanitizers] Implement GetTls on Solaris
Rainer Orth via Phabricator via llvm-commits
llvm-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 llvm-commits
mailing list