[PATCH] D70398: [lld] Support copy relocations for TLS symbols

James Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 21 16:56:12 PST 2019


jrtc27 abandoned this revision.
jrtc27 added a comment.

In D70398#1756018 <https://reviews.llvm.org/D70398#1756018>, @MaskRay wrote:

> In D70398#1755440 <https://reviews.llvm.org/D70398#1755440>, @jrtc27 wrote:
>
> > EDIT: I see the discussion linked by MaskRay has already reached the conclusion that GCC will stop emitting these. I will file a patch for LLVM to do the same if nobody beats me to it (relaxations can come later once they are finalised).
>
>
> I wanted to do this, but you claimed this first. Go ahead:) Given the consensus that TLS copy relocation is a bad idea and newer toolchains do not need to support it, can be drop this patch?


Oh, yes, I intended to abandon this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70398





More information about the llvm-commits mailing list