[llvm-dev] LLD performance w.r.t. local symbols (and --build-id)

[Rui Ueyama via llvm-dev]

On Wed, Mar 16, 2016 at 9:05 AM, Rafael Espíndola <
rafael.espindola at gmail.com> wrote:

> On 16 March 2016 at 01:34, George Rimar <grimar at accesssoftek.com> wrote:
> > Slowdown by "[ELF] - Early continue in
> InputSectionBase<ELFT>::relocate().
> > NFC." looks wierd for me. I do not see any reasons for any impact on
> > perfomance by this change.
>
>
> I think it is just because the continue is unlikely and now there is
> an early check of offset.
>
> > Good news is that since it was NFC it can easily be reverted. But I think
> > slowdown in results is unrelative with that change and reverting will not
> > give us 2-3% boost back.
>
> I don't think we should revert it right now. There are a few big
> changes I would like to try to the relocation processing code.
>

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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20160316/dba0057c/attachment.html>