<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Mar 16, 2016 at 9:05 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 16 March 2016 at 01:34, George Rimar <<a href="mailto:grimar@accesssoftek.com">grimar@accesssoftek.com</a>> wrote:<br>
> Slowdown by "[ELF] - Early continue in InputSectionBase<ELFT>::relocate().<br>
> NFC." looks wierd for me. I do not see any reasons for any impact on<br>
> perfomance by this change.<br>
<br>
<br>
</span>I think it is just because the continue is unlikely and now there is<br>
an early check of offset.<br>
<span class=""><br>
> Good news is that since it was NFC it can easily be reverted. But I think<br>
> slowdown in results is unrelative with that change and reverting will not<br>
> give us 2-3% boost back.<br>
<br>
</span>I don't think we should revert it right now. There are a few big<br>
changes I would like to try to the relocation processing code.<br></blockquote><div><br></div><div>Agreed. Unless your code changes algorithm, you don't need to worry too much about performance fluctuations caused by that change. This may vary on compiler, compiler version, code around your change, and test cases.</div></div></div></div>