[PATCH] D116333: [RISCV] Initial support for scalable vectorization

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 4 03:20:54 PST 2022


sdesmalen added a comment.

The commit message seems wrong, because this patch is about tuning the cost-model, not about enabling scalable vectorization.



================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp:166
 
+InstructionCost RISCVTTIImpl::getCastInstrCost(unsigned Opcode, Type *Dst,
+                                               Type *Src,
----------------
You may want to add some unit tests to llvm/test/Analysis/CostModel/ for this change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116333



More information about the llvm-commits mailing list