<div dir="ltr">I'm out of office today. I'll review this in a few days. Sorry for the delay.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 6, 2015 at 3:03 PM, Rafael Ávila de Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">rafael added inline comments.<br>
<br>
================<br>
</span>Comment at: ELF/InputSection.cpp:96<br>
@@ +95,3 @@<br>
+      Target->relocateOne(<br>
+          Buf + RI.r_offset, BufEnd, Type, BaseAddr + RI.r_offset,<br>
+          Out<ELFT>::Got->getVA() + Out<ELFT>::LocalModuleTlsIndexOffset +<br>
----------------<br>
Buf + RI.r_offset is now used 3 times is this function. Please factor it into a variable above, commit that and rebase.<br>
<br>
The same is true for  BaseAddr + RI.r_offset<br>
<br>
================<br>
Comment at: ELF/OutputSections.cpp:83<br>
@@ -82,1 +82,3 @@<br>
<br>
+template <class ELFT> uint32_t GotSection<ELFT>::addLocalModuleTlsIndex() {<br>
+  Entries.push_back(nullptr);<br>
----------------<br>
Will this also be used for global tls? Change the name if so.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D14359" rel="noreferrer" target="_blank">http://reviews.llvm.org/D14359</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>