[PATCH] D25960: Read section headers upfront

[Rafael Espíndola via llvm-commits]

On 25 October 2016 at 17:04, Davide Italiano <dccitaliano at gmail.com> wrote:
> davide added a comment.
>
> If you plan to iterate on this, probably it's fine, but my reaction when I see your number is that the performance effect of the change is pretty much neutral. Sure, you have some slowdowns and some improvements, but they vary roughly between 0.1% and 0.5% , which, no matter how controlled the enviroment on which we're running tests, could be caused by other factors.

Yes, note that the objective is not to optimize in this change. The
objective is to make it easier to create synthetic InputSections. I
just wanted to make sure it was not a terrible idea from a performance
perspective.

Cheers,
Rafael