[PATCH] D116890: [RISCV] Add initial support for getRegUsageForType and getNumberOfRegisters

[Kito Cheng via Phabricator via llvm-commits]

kito-cheng marked an inline comment as done.
kito-cheng added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp:285
+  if (Ty->isVectorTy()) {
+    if (Size.isScalable())
+      return divideCeil(Size.getKnownMinValue(), RISCV::RVVBitsPerBlock);
----------------
craig.topper wrote:
> Do we need to check the V extension is enabled here?
Good catch, thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116890