[PATCH] D115757: [SLP]Generalize cost model.

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:6619
-    }
+    case Instruction::Xor:
     case Instruction::GetElementPtr: {
-      TargetTransformInfo::OperandValueKind Op1VK =
----------------
RKSimon wrote:
> Why didn't we ever use getGEPCost?
It originally used add cost for GEPs, most probably there were getGEPCost. Plus, I don't see target-specific getGEPCost implementations, looks like we have only base one.
We could try to switch to getGEPCost but it must be in a separate patch.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D115757/new/

https://reviews.llvm.org/D115757