[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