[PATCH] D38961: [PowerPC] Increase the user cost of vector instructions by their legalization cost

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 16 13:08:01 PDT 2017


echristo added inline comments.


================
Comment at: lib/Target/PowerPC/PPCTargetTransformInfo.cpp:194-197
+  if (U->getType()->isVectorTy()) {
+    // Instructions that need to be split should cost more.
+    std::pair<int, MVT> LT = TLI->getTypeLegalizationCost(DL, U->getType());
+    return LT.first * BaseT::getUserCost(U, Operands);
----------------
I have concerns about this being Power only. And to what costs we're applying this to, etc.

Thoughts?


https://reviews.llvm.org/D38961





More information about the llvm-commits mailing list