[PATCH] D18039: [ELF] - Fixed handling R_X86_64_DTPOFF64 relocation relaxation

[Rui Ueyama via llvm-commits]

ruiu added inline comments.

================
Comment at: ELF/Target.cpp:896-897
@@ -895,3 +895,4 @@
   case R_X86_64_DTPOFF32:
     relocateOne(Loc, BufEnd, R_X86_64_TPOFF32, P, SA);
     return 0;
+  case R_X86_64_DTPOFF64: {
----------------
I guess this is the same? Please inline the code here in a follow-up patch.

================
Comment at: ELF/Target.cpp:900
@@ +899,3 @@
+    uint64_t Val = SA - Out<ELF64LE>::TlsPhdr->p_memsz;
+    checkInt<64>(Val, Type);
+    write64le(Loc, Val);
----------------
Does checkInt<64> do anything? A 64-bit integer is always within 64 bits.


http://reviews.llvm.org/D18039