[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