[PATCH] D34035: [LLD][ELF] Introduce Thunk reuse compatibility

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 19 04:01:47 PDT 2017


peter.smith added a comment.

Is there any more I can do to explain this patch?

In the current Thunk implementation we only permit 1 Thunk per Target symbol and by construction all callers to that symbol are compatible with the Thunk. Once we have range extension thunks this will no longer be the case. For example we might have ARM and Thumb branches (can't use blx) to a target symbol S, these callers cannot share the same Thunk.

This patch by construction will return true for all the currently supported thunks.


https://reviews.llvm.org/D34035





More information about the llvm-commits mailing list