[PATCH] D19825: Power9 - Add exploitation of vector load and store that do not require swaps

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 6 19:01:48 PDT 2016


echristo added inline comments.

================
Comment at: lib/Target/PowerPC/PPCISelLowering.cpp:10554
@@ -10550,3 +10553,3 @@
       MVT StoreVT = VT.getSimpleVT();
-      if (Subtarget.hasVSX() && Subtarget.isLittleEndian() &&
+      if (NeedsSwapsForVSXMemOps &&
           (StoreVT == MVT::v2f64 || StoreVT == MVT::v2i64 ||
----------------
Would it make sense for this to be a subtarget feature given all of the load/store instruction changes for vectors? Though I guess you couldn't just use Subtarget.isISA3_0 anymore as the canonical "am I targeting something P9 or later".




Repository:
  rL LLVM

http://reviews.llvm.org/D19825





More information about the llvm-commits mailing list