[PATCH] D34692: [LLD][ELF] Add support for multiple passes to createThunks()

[Peter Smith via Phabricator via llvm-commits]

peter.smith updated this revision to Diff 106889.
peter.smith added a comment.

Rebased to account for r308056 and r308057. I've also updated mergeThunks() to account for Igor's test case (thanks very much Igor!). I've taken the simple solution of making sure that the empty ThunkSections are removed from ThunkSections and NewThunkSections, a test case derived from Igor's has been added to make sure this doesn't regress. My expectation is that the vast majority of Thunks will be added in the first pass so the lack of pre-created sections in later passes shouldn't make a significant difference.


https://reviews.llvm.org/D34692

Files:
  ELF/Relocations.cpp
  ELF/Relocations.h
  ELF/Thunks.cpp
  ELF/Thunks.h
  ELF/Writer.cpp
  test/ELF/arm-thumb-condbranch-thunk.s
  test/ELF/arm-thumb-thunk-empty-pass.s
  test/ELF/arm-thunk-multipass.s
  test/ELF/arm-thunk-re-add.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34692.106889.patch
Type: text/x-patch
Size: 30533 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170717/7fec0145/attachment.bin>