[PATCH] D120059: [sanitizer_common] Support Solaris < 11.4 in GetStaticTlsBoundary

Rainer Orth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 6 07:28:05 PDT 2022


ro added a comment.
Herald added a subscriber: StephenFan.
Herald added a project: All.

In D120059#3338258 <https://reviews.llvm.org/D120059#3338258>, @ro wrote:

> Abandoned unless absolutely necessary for Illumos support.  As discussed, a way better alternative would be for them to implement `dlpi_tls_modid` or at least provide a way to distuingish Illumos from Solaris as outlined in Issue #53919.

Current `compiler-rt` was just integrated into GCC trunk, which broke the build on Solaris 11.3 for the lack of `dlpi_tls_modid`.  S11.3 is still supported in GCC (and will remain so for quite some time), so I guess the best option forward is to reactivate this patch? @MaskRay?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120059



More information about the llvm-commits mailing list