[PATCH] D67284: [LLD][ELF][ARM] Implement --fix-cortex-a8 to fix erratum 657417

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 13 09:35:20 PDT 2019


peter.smith updated this revision to Diff 220127.
peter.smith marked 3 inline comments as done.
peter.smith added a comment.

Updated diff for latest suggestions. Main changes:

- Simplify the mapping symbol calculations
- Cache the result of searching for the relocation
- Fix a bug when determining whether a patch could reach outside its section.
- Update comments and fix white space in tests.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67284/new/

https://reviews.llvm.org/D67284

Files:
  ELF/AArch64ErrataFix.cpp
  ELF/ARMErrataFix.cpp
  ELF/ARMErrataFix.h
  ELF/CMakeLists.txt
  ELF/Config.h
  ELF/Driver.cpp
  ELF/Options.td
  ELF/Writer.cpp
  test/ELF/arm-fix-cortex-a8-blx.s
  test/ELF/arm-fix-cortex-a8-nopatch.s
  test/ELF/arm-fix-cortex-a8-plt.s
  test/ELF/arm-fix-cortex-a8-recognize.s
  test/ELF/arm-fix-cortex-a8-thunk.s
  test/ELF/arm-fix-cortex-a8-toolarge.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67284.220127.patch
Type: text/x-patch
Size: 48401 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190913/f94c6900/attachment.bin>


More information about the llvm-commits mailing list