<div dir="ltr">LGTM, but I would expand on the comment a bit:<div><div>     // The ABI says: "It is suggested that it be emitted in the same COMDAT group</div><div>     // as the associated data object." In practice, this doesn't work for non-ELF object formats, so only do it for ELF.</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 3, 2015 at 11:31 AM, Rafael 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">I fully trust Reid review on this, so OK with me.<br>
<div class="HOEnZb"><div class="h5"><br>
On 3 September 2015 at 14:04, Yaron Keren <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>> wrote:<br>
> <a href="http://llvm.org/pr23472" rel="noreferrer" target="_blank">http://llvm.org/pr23472</a><br>
><br>
> As suggested by Reid, OK to commit?<br>
><br>
</div></div></blockquote></div><br></div>