[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