<p dir="ltr">Lgtm </p>
<div class="gmail_quote">On Apr 25, 2016 3:14 PM, "Simon Atanasyan" <<a href="mailto:simon@atanasyan.com">simon@atanasyan.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">atanasyan added inline comments.<br>
<br>
================<br>
Comment at: ELF/OutputSections.cpp:149<br>
@@ -148,3 +148,3 @@<br>
     return false;<br>
-  Sym.GlobalDynIndex = Target->GotHeaderEntriesNum + Entries.size();<br>
+  Sym.GlobalDynIndex = Entries.size();<br>
   // Global Dynamic TLS entries take two GOT slots.<br>
----------------<br>
rafael wrote:<br>
> Won't this produce the wrong result for mips now?<br>
Now LLD does not support MIPS TLS relocations. So this code does not make worse anything.<br>
<br>
I have another couple of patches to support MIPS TLS. As an announce - it might sound like a joke but for GOT entries related to TLS relocations MIPS ABI uses almost regular approach with creation of dynamic relocations for each GOT enty etc. But ABI requires to separate these 'regular' TLS related entries from MIPS specific local/global parts of GOT :)<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D19465" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19465</a><br>
<br>
<br>
<br>
</blockquote></div>