<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Nov 14, 2017 at 2:00 AM, 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="">James Henderson via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> writes:<br>
<br>
> jhenderson added a comment.<br>
><br>
> Given that the issue with not combining sections was purely restricted to .debug_str, which only ever has an entsize of 1, would it not make more sense to not combine the InputSections for different entsize values?<br>
><br>
> I'm wary about removing the SHF_MERGE flag, because I'm pretty sure mergeable sections of different sizes are not that rare, although they do not typically have the same name, so maybe it's okay.<br>
<br>
</span>I agree. We should merge SHF_MERGE sections in pretty much the same way<br>
we do for regular (non -r) output. We then just assign each<br>
MergeSyntheticSection to its own OutputSection and keep the SHF_MERGE<br>
flag.<br></blockquote><div><br></div><div>It sounds like a plan. I believe it can naturally be implemented. </div></div></div></div>