<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, May 24, 2017 at 3:36 PM, Rafael Avila de Espindola <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="">Rui Ueyama via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> writes:<br>
<br>
> ruiu added a comment.<br>
><br>
> In general, I think we pass through all input sections to an output when the -r option is given. But in this patch you deduplicates comdat groups. Do you need that?<br>
<br>
</span>While it should be legal to have two groups with the same signature in a .o<br>
file, it seems better to drop the second one, no?<br></blockquote><div><br></div><div>Yes, but I wondered if we could make this patch simpler if we don't dedup comdat groups.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
For what it is worth, both bfd and gold drop it.<br>
<br>
Taking a look at the patch.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div></div>