[PATCH] D63840: [ARM] Add support for MVE pre and post inc loads and stores.
Dave Green via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 15 06:49:42 PDT 2019
dmgreen updated this revision to Diff 209840.
dmgreen added a comment.
The original version wasn't getting alignment correct in some cases. For little endian which instruction we choose has more to do with alignment and offset than type being loaded. I've rewritten parts of this to, at least for LE, get this hopefully more correct. I've tried to add BE too, but not added any tests for that yet. We still have a task to sort out BE properly.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63840/new/
https://reviews.llvm.org/D63840
Files:
llvm/include/llvm/Target/TargetSelectionDAG.td
llvm/lib/Target/ARM/ARMISelDAGToDAG.cpp
llvm/lib/Target/ARM/ARMISelLowering.cpp
llvm/lib/Target/ARM/ARMInstrMVE.td
llvm/test/CodeGen/Thumb2/mve-ldst-postinc.ll
llvm/test/CodeGen/Thumb2/mve-ldst-preinc.ll
llvm/test/CodeGen/Thumb2/mve-ldst-regimm.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63840.209840.patch
Type: text/x-patch
Size: 61319 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190715/ccab7939/attachment.bin>
More information about the llvm-commits
mailing list