[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