[PATCH] D67327: [ARM] Fix for MVE narrow load/store stack accesses

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 17 02:38:58 PDT 2019


dmgreen added a comment.

The AddrModeT2_i7, etc can still be generated from the non-widening forms of the MVE loads/stores (as they accept a GPR not a tGPR they can take SP too). I don't think the narrowing loads (the ones that cannot accept a stack pointer) will be generated any more, but I'd prefer to fix these cases nonetheless. They may change in the future and these failures are silent until the function gets complex enough to trigger them, so can be difficult to test for.


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

https://reviews.llvm.org/D67327





More information about the llvm-commits mailing list