[PATCH] D91605: [sanitizers] Implement GetTls on Solaris
Rainer Orth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Feb 14 06:26:27 PST 2022
ro updated this revision to Diff 408389.
ro added a subscriber: nikic.
ro added a comment.
Since the build-time check for the `ld -z relax=transtls` option was met with massive resistance and was only necessary for Illumos anyway, this revision just uses it unconditionally.
Tested on `amd64-pc-solaris2.11` and `sparcv9-sun-solaris2.11`.
The Illumos community will have to provide a way to somehow distinguish the two OSes. The configure triple is still the same, and the `uname -o` output (`Solaris` vs. `illumos` seems to be hardcoded in the `uname` command and not be available programmatically.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91605/new/
https://reviews.llvm.org/D91605
Files:
clang/lib/Driver/ToolChains/Solaris.cpp
compiler-rt/lib/sanitizer_common/sanitizer_linux_libcdep.cpp
compiler-rt/lib/sanitizer_common/sanitizer_solaris.h
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91605.408389.patch
Type: text/x-patch
Size: 5378 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220214/e65dc870/attachment-0001.bin>
More information about the cfe-commits
mailing list