[PATCH] D56779: [ELF][X86_64] Fix corrupted LD -> LE optimization for TLS without PLT

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 16 11:06:27 PST 2019


ruiu added inline comments.


================
Comment at: ELF/Arch/X86_64.cpp:267-275
   // Convert
-  //   leaq bar at tlsld(%rip), %rdi
-  //   callq __tls_get_addr at PLT
+  //   leaq bar at tlsld(%rip), %rdi     # 48 8d 3d <Loc>
+  //   callq __tls_get_addr at PLT       # e8 <disp32>
   //   leaq bar at dtpoff(%rax), %rcx
   // to
   //   .word 0x6666
   //   .byte 0x66
----------------
Now this comment is at a wrong place? It looks like this comment explains the entire function, but I believe now this comment explains on the first "if" condition.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56779/new/

https://reviews.llvm.org/D56779





More information about the llvm-commits mailing list