[PATCH] D10930: [ELF/AArch64] Add support for external TLS variables
Rui Ueyama
ruiu at google.com
Fri Jul 3 09:26:18 PDT 2015
ruiu accepted this revision.
This revision is now accepted and ready to land.
LGTM
================
Comment at: lib/ReaderWriter/ELF/AArch64/AArch64RelocationPass.cpp:257-258
@@ -257,3 +256,4 @@
std::error_code handleGOTTPREL(const Reference &ref) {
- if (isa<DefinedAtom>(ref.target()))
+ if (isa<DefinedAtom>(ref.target()) ||
+ isa<SharedLibraryAtom>(ref.target()))
const_cast<Reference &>(ref).setTarget(getGOTTPREL(ref.target()));
----------------
Maybe these two lines can fit in 80 columns?
http://reviews.llvm.org/D10930
More information about the llvm-commits
mailing list