[PATCH] D33041: [ELF] - Set DF_STATIC_TLS flag for x86_64 target.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 4 00:56:01 PST 2019


grimar added a comment.

In D33041#1382592 <https://reviews.llvm.org/D33041#1382592>, @emaste wrote:

> FreeBSD patch adding initial exec TLS mode for dynamically loaded shared objects: https://reviews.freebsd.org/D19072. Can we revisit this lld patch?


I'll take a look tomorrow. Seems FreeBSD is going to use it for both x86_64 and i386 targets for start? That means we also might want to revisit D32354 <https://reviews.llvm.org/D32354>.
It was reverted.

FTR and to answer the comment from freebsd.org page above, which was
"They did not stated why DF_STATIC_TLS was considered as 'not needed'.":

Rafael's answer in the mail list that time was:

> I just cloned musl and DF_STATIC_TLS is not used anywhere.
> 
> glibc used to use it, but doesn't since 2002
>  (2430d57a13f4f10312e13c58962cd9104e6428fd).
> 
> It is also not used by the freebsd one.




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

https://reviews.llvm.org/D33041





More information about the llvm-commits mailing list