[PATCH] D67622: [LLD][AARCH64] Small refactor of AArchErrataFix to match changes in ARMErrataFix NFC.

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 16 07:51:20 PDT 2019


peter.smith created this revision.
peter.smith added reviewers: ruiu, MaskRay, grimar.
Herald added subscribers: dmgreen, kristof.beyls, arichardson, emaste.
Herald added a reviewer: espindola.

D67284 <https://reviews.llvm.org/D67284> introduced ARMErrataFix.cpp which was derived from AArch64ErrataFix.cpp. There were some useful refactoring changes made to ARMErrataFix.cpp made as part of the review. This change applies the relevant changes back to AArch64ErrataFix.cpp. Main changes are:

- Old style variable names in comments like IS, are now new style isec.
- Simplify init() collection of mappingSymbols to always start with a code mapping symbol.
- Simplify logic in mergeCmp().
- Fix one 80 column overflow caused by IS -> isec transformation.


https://reviews.llvm.org/D67622

Files:
  ELF/AArch64ErrataFix.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67622.220342.patch
Type: text/x-patch
Size: 5244 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190916/eebd91af/attachment.bin>


More information about the llvm-commits mailing list