[PATCH] D41246: [LLD][ELF] Optimize Arm PLT entries
Rafael Avila de Espindola via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 14 19:55:03 PST 2017
Peter Smith via Phabricator <reviews at reviews.llvm.org> writes:
> peter.smith added a comment.
>
> In https://reviews.llvm.org/D41246#955551, @pcc wrote:
>
>> Instead of making the behaviour conditional on the flag, could we always emit the long PLT header and then conditionally emit short or long PLTs depending on what the offset is?
>
>
> That sounds like it should work given that the entries are the same
> size.
The advantage of the smaller sequence is just that it has 1 load instead
of 2, right? If so please add a comment saying that.
Cheers,
Rafael
More information about the llvm-commits
mailing list