[PATCH] D17529: ELF: Implement ICF.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 22 18:56:34 PST 2016


On Mon, Feb 22, 2016 at 6:48 PM, Sean Silva <chisophugis at gmail.com> wrote:

> silvas added inline comments.
>
> ================
> Comment at: ELF/ICF.cpp:285
> @@ +284,3 @@
> +
> +  const Elf_Shdr *RA = A->RelocSections[0];
> +  const Elf_Shdr *RB = B->RelocSections[0];
> ----------------
> Why only the first element of RelocSections?
>

Look for "nothing wrong with sections with more than one relocation" in my
patch.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160222/736744c6/attachment.html>


More information about the llvm-commits mailing list