<div dir="ltr">I don't think this comments add much value to the code because it's already commented well. We have comments just above these new comments to show the final results of the calculations, so it is very easy to understand the intention of the code. If you get what we want to get here, it's in my opinion very easy to understand the code as well.<div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 30, 2015 at 2:11 AM, George Rimar via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: grimar<br>
Date: Fri Oct 30 04:11:12 2015<br>
New Revision: 251692<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=251692&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=251692&view=rev</a><br>
Log:<br>
Comments about how values were calculated for functions below were added.<br>
1. X86_64TargetInfo::writePltZeroEntry.<br>
2. X86_64TargetInfo::writePltEntry.<br>
<br>
Modified:<br>
    lld/trunk/ELF/Target.cpp<br>
<br>
Modified: lld/trunk/ELF/Target.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Target.cpp?rev=251692&r1=251691&r2=251692&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Target.cpp?rev=251692&r1=251691&r2=251692&view=diff</a><br>
==============================================================================<br>
--- lld/trunk/ELF/Target.cpp (original)<br>
+++ lld/trunk/ELF/Target.cpp Fri Oct 30 04:11:12 2015<br>
@@ -228,6 +228,10 @@ void X86_64TargetInfo::writePltZeroEntry<br>
       0xff, 0x25, 0x00, 0x00, 0x00, 0x00, // jmp *GOT+16(%rip)<br>
       0x0f, 0x1f, 0x40, 0x00              // nopl 0x0(rax)<br>
   };<br>
+  // 1. NextPC = PltEntryAddr + 6,<br>
+  // GotEntryAddr - NextPC + 8 = GotEntryAddr - PltEntryAddr + 2.<br>
+  // 2. NextPC = PltEntryAddr + 6 + 6,<br>
+  // GotEntryAddr - NextPC + 16 = GotEntryAddr - PltEntryAddr + 4.<br>
   memcpy(Buf, PltData, sizeof(PltData));<br>
   write32le(Buf + 2, GotEntryAddr - PltEntryAddr + 2); // GOT+8<br>
   write32le(Buf + 8, GotEntryAddr - PltEntryAddr + 4); // GOT+16<br>
@@ -243,6 +247,12 @@ void X86_64TargetInfo::writePltEntry(uin<br>
   };<br>
   memcpy(Buf, Inst, sizeof(Inst));<br>
<br>
+  // 1. NextPC = PltEntryAddr + 6,<br>
+  // GotEntryAddr - NextPC = GotEntryAddr - PltEntryAddr - 6.<br>
+  // 2. Index is just and index of PLT record.<br>
+  // 3. NextPC = PltEntryAddr + 6 + 5 + 5 = PltEntryAddr + 16,<br>
+  // PltEntryAddr - Index * PltEntrySize - PltZeroEntrySize - NextPC =<br>
+  // -Index * PltEntrySize - PltZeroEntrySize - 16.<br>
   write32le(Buf + 2, GotEntryAddr - PltEntryAddr - 6);<br>
   write32le(Buf + 7, Index);<br>
   write32le(Buf + 12, -Index * PltEntrySize - PltZeroEntrySize - 16);<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div></div>