[PATCH] D30680: new method TargetTransformInfo::supportsVectorElementLoadStore() for LoopVectorizer

Jonas Paulsson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 15 00:30:12 PDT 2017


jonpa added a comment.

In https://reviews.llvm.org/D30680#700694, @anemet wrote:

> Why aren't you putting this in TTI.getOperandsScalarizationOverhead?


I don't see how that would work, because this is dependent on the instruction opcode. It is only relevant if it is a store or a load, and only for the store case are the operands checked for extracts. Are you suggesting adding an extra parameter to that function (opcode)?


https://reviews.llvm.org/D30680





More information about the llvm-commits mailing list