[PATCH] D31274: Remove an optimization to set the TLS module index to 1.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 22 21:22:00 PDT 2017


ruiu created this revision.
Herald added subscribers: emaste, rengolin, aemerson.

There is a piece of code to set TLS module index to 1 only when
it is ARM and we are linking an executable (as opposed to DSOs).
I'm not sure how useful it is. If this is just to save one dynamic
relocation at load-time, I want to remove this to simplify this code.
We'll probably be able to restore this functionality in future once
I'm done with refactoring.


https://reviews.llvm.org/D31274

Files:
  lld/ELF/Relocations.cpp
  lld/ELF/Target.cpp
  lld/test/ELF/arm-tls-norelax-gd-le.s
  lld/test/ELF/arm-tls-norelax-ld-le.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D31274.92757.patch
Type: text/x-patch
Size: 4176 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170323/1d9ec563/attachment.bin>


More information about the llvm-commits mailing list