[PATCH] D86776: [ARM]{MVE] Enable MVE gathers and scatters by default

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 28 08:13:46 PDT 2020


dmgreen accepted this revision.
dmgreen added a comment.
This revision is now accepted and ready to land.

I asked Anna to put this patch together so that we had it when we needed it. I've been testing things out and I don't see any real reason not to go for it now though. Performance looks OK, if a bit conservative now that we have the higher costs. And I've not found any correctness issues. I've been trying any code I can find with a forced vector user cost and not run into any problems.

I think this is good to go.

LGTM. Thanks!


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

https://reviews.llvm.org/D86776



More information about the llvm-commits mailing list