This is an archive of the discontinued LLVM Phabricator instance.

Add support for TLS IEGOT relocations
AbandonedPublic

Authored by sidneym on Dec 10 2019, 8:18 AM.

Details

Summary

Adds support for IEGOT relocations.

The patch includes references to:

https://reviews.llvm.org/D71069
https://reviews.llvm.org/D71143

The file referenced in Inputs is from: https://reviews.llvm.org/D71143

I can merge the file in Inputs into the testscases but I think the duplication would be an issue.

Diff Detail

Event Timeline

sidneym created this revision.Dec 10 2019, 8:18 AM
Herald added a project: Restricted Project. · View Herald Transcript
MaskRay added inline comments.Dec 10 2019, 9:09 AM
lld/test/ELF/hexagon-tls-ie-got.s
1

Probably delete -got from the filename. Initial Exec uses GOT on all targets.

3

You can place the definitions in the same file. A definition in a DSO is by default preemptible (unless marked as non-STV_DEFAULT, or -Bsymbolic)

See ppc32-tls-ie.s for an example.

7

In the previous review, you mentioned that Hexagon does not support TLS relaxations. It is worth testing this property explicitly. See the ld.lld %t.o -o %t and LE: LE-NEXT: part of ppc32-tls-ie.s.

sidneym abandoned this revision.Jan 14 2020, 1:09 PM
sidneym marked 2 inline comments as done.