[PATCH] D36749: [LLD][ELF][AArch64] Complete implementation of -fix-cortex-a53-843419

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 4 09:42:29 PST 2017


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

Rebased after changes to https://reviews.llvm.org/D36749.

- We no longer initialize the mapping symbols in the constructor as we may be constructed but not used.
- If a patch is inserted we must update addresses again for thunks.

Still todo:

- The implementation assumes that no OutputSection is > 128 Mb. I've encountered at least one real world program (gitit) that has a 180 Mb .text section, so I'll need to fix that particular fixme.


https://reviews.llvm.org/D36749

Files:
  ELF/AArch64ErrataFix.cpp
  ELF/AArch64ErrataFix.h
  ELF/Writer.cpp
  test/ELF/aarch64-cortex-a53-843419-address.s
  test/ELF/aarch64-cortex-a53-843419-recognize.s
  test/ELF/aarch64-cortex-a53-843419-thunk.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D36749.125357.patch
Type: text/x-patch
Size: 45388 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171204/09846933/attachment.bin>


More information about the llvm-commits mailing list