[PATCH] D31605: [LLD][ELF][ARM] Rename ARM Thunks in anticipation of Range Thunks

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 3 09:14:30 PDT 2017


peter.smith created this revision.
Herald added subscribers: rengolin, aemerson.

The existing names for the ARM and Thumb Thunks highlight their current use as interworking Thunks. These Thunks can also be used for range extension Thunks where there is no state change. This change makes it clearer that the destination can be either ARM or Thumb.

I've now got a complete range extension implementation and tests, I'll be sending this for review tomorrow. I've split it into 10 or so smaller patches and 2 patches to add tests as it is a bit too big to review in one go. Do you have any preference for how I should submit it? My current plan is to make a Phabricator review for each of them and then write an RFC to llvm-dev to explain how they all fit together.


https://reviews.llvm.org/D31605

Files:
  ELF/Thunks.cpp
  test/ELF/arm-thumb-interwork-shared.s
  test/ELF/arm-thumb-interwork-thunk.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D31605.93872.patch
Type: text/x-patch
Size: 18539 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170403/ed5e7e4c/attachment.bin>


More information about the llvm-commits mailing list