[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