[PATCH] D11612: [lld][ELF2] Apply relocations.

Rui Ueyama ruiu at google.com
Wed Jul 29 17:58:13 PDT 2015


ruiu added a comment.

IIUC, doing GOT/PLT and applying relocations in one pass is just merging
multiple loops to one loop, and the total number of computation we need
to do for GOT/PLT and applying relocations wouldn't change that much.
Because relocations are continuous in memory and they are fixed in size,
iterating over them should be very fast, so I don't see loop-merging would
make things noticeably faster (that's what I observed in COFF.)

So I feel like it's a premature optimization. I'd like to keep it really simple and
straightforward as possible at this moment. We'll be able to do optimization
later.

Rafael, I'd like to hear from you since this is your idea.


http://reviews.llvm.org/D11612







More information about the llvm-commits mailing list