[PATCH] D62967: [ARM] Add MVE addressing to isLegalT2AddressImmediate

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 7 05:20:07 PDT 2019


dmgreen updated this revision to Diff 203538.
dmgreen added a comment.

Cleanup and added mve (nofp) to the tests.

Worth noting that I'm pretty sure we do have sensible vector load instructions for floats without mve.fp (they are the same as integer loads). We don't generate them at the moment, and this more accurately models the mess that we do create (i.e. it's expensive).


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62967/new/

https://reviews.llvm.org/D62967

Files:
  llvm/lib/Target/ARM/ARMISelLowering.cpp
  llvm/test/Analysis/CostModel/ARM/gep.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D62967.203538.patch
Type: text/x-patch
Size: 18881 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190607/1ddd5347/attachment-0001.bin>


More information about the llvm-commits mailing list