Page MenuHomePhabricator

[LLD][ELF] Allow multiple thunks to be added for a symbol.
ClosedPublic

Authored by peter.smith on Jun 8 2017, 7:23 AM.

Details

Summary

This change permits there to be more than one thunk to be associated with a symbol. For interworking thunks we only require one thunk, but range
extension thunks may require more than one.

Diff Detail

Repository
rL LLVM

Event Timeline

peter.smith created this revision.Jun 8 2017, 7:23 AM
ruiu added inline comments.Jun 8 2017, 10:58 AM
ELF/Relocations.cpp
1059–1060 ↗(On Diff #101917)

Can you return {ET, false}?

1064 ↗(On Diff #101917)

If so, you can move the definition of T here.

Thanks for the comments. I've updated the diff to return {ET, false} as suggested.

Is there anything more I can do to move this patch along?

This is patch 2/11 of range thunks. Although it can be independently applied from patch 1.

This revision was automatically updated to reflect the committed changes.