[PATCH] D66222: [ARM] Fix alignment checks for BE VLDRH

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 14 08:36:23 PDT 2019


dmgreen created this revision.
dmgreen added reviewers: t.p.northover, samparker, SjoerdMeijer, simon_tatham, ostannard.
Herald added subscribers: hiraditya, kristof.beyls, javed.absar.
Herald added a project: LLVM.

We need to allow any alignment at least 2, not just exactly 2, so that the big endian loads and stores can be selected successfully. I've also added extra BE testing for the load and store tests.

Thanks to Oliver for the report.


https://reviews.llvm.org/D66222

Files:
  llvm/lib/Target/ARM/ARMInstrMVE.td
  llvm/test/CodeGen/Thumb2/mve-ldst-offset.ll
  llvm/test/CodeGen/Thumb2/mve-ldst-postinc.ll
  llvm/test/CodeGen/Thumb2/mve-ldst-preinc.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D66222.215130.patch
Type: text/x-patch
Size: 39600 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190814/af059090/attachment.bin>


More information about the llvm-commits mailing list