<div dir="ltr">This is a good improvement. <div><br></div><div>LGTM.</div><div><br></div><div>Thanks!</div><div><br></div><div>-eric<br><br><div class="gmail_quote">On Mon, Mar 23, 2015 at 1:56 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">Eric had so objections to the way unique sections were handled. I<br>
decided then to split this patch in two.<br>
<br>
The first one (attached) is just the refactoring on how different<br>
passes get a label a the end of a section.<br>
<br>
OK?<br>
<br>
<br>
On 20 March 2015 at 16:57, Pete Cooper <<a href="mailto:peter_cooper@apple.com" target="_blank">peter_cooper@apple.com</a>> wrote:<br>
><br>
>> On Mar 20, 2015, at 1:51 PM, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>> wrote:<br>
>><br>
>>>> Sounds like a nice cleanup to me, although I don't know this code<br>
>>>> well enough to confirm.<br>
>><br>
>> All dependencies are in.<br>
>><br>
>> A rebased patch is attached.<br>
>><br>
>> OK?<br>
> Hi Rafael<br>
><br>
> Please add a comment to say that EmitFunctionHeader() returns whether the section is unique.  After that LGTM.<br>
><br>
> Cheers,<br>
> Pete<br>
>><br>
>> Cheers,<br>
>> Rafael<br>
>> <t.patch><br>
><br>
</blockquote></div></div></div>