[PATCH] D19825: Power9 - Add exploitation of vector load and store that do not require swaps
Kit Barton via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 11 11:30:33 PDT 2016
kbarton requested changes to this revision.
kbarton added a comment.
This revision now requires changes to proceed.
As Eric suggested, remove the old patterns to ensure LXVX/STXVX instructions are generated, instead of using AddedComplexity.
================
Comment at: lib/Target/PowerPC/PPCInstrVSX.td:2161
@@ +2160,3 @@
+
+ let AddedComplexity = 500 in {
+ def : Pat<(v2f64 (load xoaddr:$src)), (LXVX xoaddr:$src)>;
----------------
echristo wrote:
> Can we just exclude the other patterns? We don't ever want them to show up do we?
I agree, it would be better to just use the LXVX and STXVX instructions when they are available.
So, we would need to remove the previous patterns for ISA3_0, but keep them for older ISAs.
Repository:
rL LLVM
http://reviews.llvm.org/D19825
More information about the llvm-commits
mailing list