[PATCH] D152001: [RISCV][SLP] Inflate insert/extract costs on very small vectors

[ShihPo Hung via Phabricator via llvm-commits]

arcbbb added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp:1526
+    BaseCost += 2;
+
   return BaseCost + SlideCost;
----------------
Is it needed for CostKind == TTI::TCK_CodeSize ?

I was thinking the total cost was composed of vmv instruction (BaseCost) and vslide instruction (SlideCost).
IIUC, BaseCost includes vector-scalar communication cost, and the addend `2` here accounts for this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152001