[PATCH] D61638: [LV] Move getScalarizationOverhead and vector call cost computations to CM. (NFC)

[Renato Golin via Phabricator via llvm-commits]

rengolin accepted this revision.
rengolin added a comment.
This revision is now accepted and ready to land.

looks obviously good to me. thanks!



================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:6043
     CallInst *CI = cast<CallInst>(I);
-    unsigned CallCost = getVectorCallCost(CI, VF, TTI, TLI, NeedToScalarize);
+    unsigned CallCost =
+        getVectorCallCost(CI, VF, NeedToScalarize);
----------------
spurious new line?


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:6647
       bool NeedToScalarize;
-      unsigned CallCost = getVectorCallCost(CI, VF, *TTI, TLI, NeedToScalarize);
+      unsigned CallCost =
+          CM.getVectorCallCost(CI, VF, NeedToScalarize);
----------------
same here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61638