[PATCH] D34689: [LLD][ELF] Pre-create ThunkSections at Target specific intervals
Peter Smith via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 26 05:34:26 PDT 2017
peter.smith updated this revision to Diff 120398.
peter.smith added a comment.
Strictly speaking we don't need the ThunkSectionSize variable. I think it is worth having ThunkSectionSpacing less than the maximum branch range as it makes it more likely that all the thunks can be created in one pass. Does the value we reduce ThunkSectionSpacing by need to be as large as 16,384 Thunks, most likely not. It is a very conservative upper bound. I can change it but it will mean updating some of the tests in later approved reviews so I'd prefer not to do it here.
What I have done is to merge ThunkSectionSize into ThunkSectionSpacing by including the ThunkSectionSize in ThunkSectionSpacing, I've updated the comment to try and explain it a bit better as well.
https://reviews.llvm.org/D34689
Files:
ELF/Arch/ARM.cpp
ELF/Relocations.cpp
ELF/Relocations.h
ELF/SyntheticSections.cpp
ELF/Target.h
test/ELF/arm-thumb-thunk-symbols.s
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34689.120398.patch
Type: text/x-patch
Size: 9281 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171026/99591057/attachment.bin>
More information about the llvm-commits
mailing list