[PATCH] D67005: [ARM] MVE: isLegalMaskedLoad
Dave Green via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 2 09:05:58 PDT 2019
dmgreen added a comment.
First things first, I think we need to fix the saving/loading of predicates, which isn't going to be pretty but is I think we have to do. I'll put together a patch for that.
Then it's probably best to re-enable masked loads/stores but behind an option. That'll let you go ahead with your testing, without making everything too inefficient or broken. Will that work for you? So long as you are not needing narrowing/widening masked loads/stores, it should look OK.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67005/new/
https://reviews.llvm.org/D67005
More information about the llvm-commits
mailing list