This is an archive of the discontinued LLVM Phabricator instance.

[ELF/AArch64] Add support for external TLS variables
ClosedPublic

Authored by zatrazz on Jul 3 2015, 8:11 AM.

Details

Summary

This patch add support to shared library TLS variable in AArch64RelocationPass
handleGOTTPREL creation and also reword the method comment.

Diff Detail

Event Timeline

zatrazz updated this revision to Diff 29023.Jul 3 2015, 8:11 AM
zatrazz retitled this revision from to [ELF/AArch64] Add support for external TLS variables.
zatrazz updated this object.
zatrazz added reviewers: ruiu, shankar.easwaran.
zatrazz added a project: lld.
zatrazz added subscribers: llvm-commits, lld.
ruiu accepted this revision.Jul 3 2015, 9:26 AM
ruiu edited edge metadata.

LGTM

lib/ReaderWriter/ELF/AArch64/AArch64RelocationPass.cpp
257–258

Maybe these two lines can fit in 80 columns?

This revision is now accepted and ready to land.Jul 3 2015, 9:26 AM
zatrazz closed this revision.Jul 9 2015, 11:53 AM