<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 22, 2016 at 6:56 PM, Rui Ueyama <span dir="ltr"><<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Mon, Feb 22, 2016 at 6:48 PM, Sean Silva <span dir="ltr"><<a href="mailto:chisophugis@gmail.com" target="_blank">chisophugis@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">silvas added inline comments.<br>
<br>
================<br>
Comment at: ELF/ICF.cpp:285<br>
@@ +284,3 @@<br>
+<br>
+  const Elf_Shdr *RA = A->RelocSections[0];<br>
+  const Elf_Shdr *RB = B->RelocSections[0];<br>
----------------<br>
Why only the first element of RelocSections?<br></blockquote><div><br></div></span><div>Look for "<span style="color:rgb(0,0,0)">nothing wrong with sections with more than one relocation" in my patch.</span> </div></div></div></div></blockquote><div><br></div><div>Ah. Should we use <a href="http://llvm.org/docs/doxygen/html/classllvm_1_1TinyPtrVector.html">http://llvm.org/docs/doxygen/html/classllvm_1_1TinyPtrVector.html</a> for RelocSections then? That will save 3 pointers (on 64-bit, 24 bytes) off of InputSectionBase, which is fairly substantial considering that before this patch, on 64-bit InputSectionBase is 88 bytes IIRC.</div><div><br></div><div>-- Sean Silva </div></div><br></div></div>