[PATCH] D15060: [ELF] - Implemented some GD, LD and IE TLS access models for x86 target.

[George Rimar via llvm-commits]

grimar added inline comments.

================
Comment at: ELF/Target.cpp:338
@@ -330,1 +337,3 @@
     break;
+  case R_386_TLS_GD:
+  case R_386_TLS_LDM:
----------------
ruiu wrote:
> Move this after R_386_PC32.
Done.

================
Comment at: ELF/Target.cpp:341
@@ +340,3 @@
+  case R_386_TLS_TPOFF: {
+    uint64_t Offset = SA - Out<ELF32LE>::Got->getVA() -
+                      Out<ELF32LE>::Got->getNumEntries() * 4;
----------------
ruiu wrote:
> Offset -> V for consistency.
Done.

================
Comment at: ELF/Target.cpp:347
@@ +346,3 @@
+  }
+  case R_386_TLS_LDO_32:
+    write32le(Loc, SA);
----------------
ruiu wrote:
> Move this before R_386_TLS_PC32 to sort.
There is no R_386_TLS_PC32 here, I think you meant before R_386_TLS_LE. Moved there.


Repository:
  rL LLVM

http://reviews.llvm.org/D15060