This patch enables MVE gather/scatters by default.
The respective command line argument can then be removed from all tests for gather/scatters, and a few other tests have to be adapted.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Before we flip the switch, can you give an impression of the performance impact of this? Does this not regress cases, is it overall a win, etc.?
Comment Actions
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!