[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