<div dir="ltr">Much nicer, thanks Rafael.<br><br>Go ahead for the non-MCSectionELF cases to add a /* !unique */ to the false operand to MCSection if you wouldn't mind. Otherwise OK.<div><br></div><div>Thanks!</div><div><br></div><div>-eric</div></div><br><div class="gmail_quote">On Mon, Mar 23, 2015 at 5:15 PM Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 23 March 2015 at 17:03, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:<br>
> This is a good improvement.<br>
><br>
> LGTM.<br>
<br>
Thanks for the quick review.<br>
<br>
Attached is an alternative patch for the rest. It moves the Unique bit<br>
to MCSection and ensure unique sections are ended when we switch away<br>
from them.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div>