<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 2, 2015 at 7:48 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"><span class="">On 1 February 2015 at 19:58, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
> Might be easier to review in Phab.<br>
><br>
> I get some/most of it - but not sure about some other bits. Like why did the<br>
> "if (!Section) {" section move from before to after the sort ("// Sort the<br>
> symbols by offset within the section.")?<br>
<br>
</span>I created <a href="http://reviews.llvm.org/D7347" target="_blank">http://reviews.llvm.org/D7347</a>.<br></blockquote><div><br></div><div>Thanks - that helps a lot. Replied with a few comments on the review.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
The code is simpler with the move since that is a simple case and<br>
becomes just if(..) {...; continue;}<br></blockquote><div><br>Ah, I can see it now - thanks!<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div></div>