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

[Peter Smith via Phabricator via llvm-commits]

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

Thanks for the comments. Updated diff to address George's comments. Changes include:

- remove uses of this
- inline addend
- add braces where needed
- change parameter name from b to s


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.219499.patch
Type: text/x-patch
Size: 48410 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190910/aa305f43/attachment.bin>