[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