[PATCH] D125856: [RISCV] Add cost model for SK_Reverse

[LiqinWeng via Phabricator via llvm-commits]

Miss_Grape added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp:272
 
+InstructionCost RISCVTTIImpl::getMemoryOpCost(unsigned Opcode, Type *Ty,
+                                              MaybeAlign Alignment,
----------------
craig.topper wrote:
> This doesn't seem to go with the title of this patch.
To ensure that the Load/store's InstructionCost‘s state is set to Valid， when  VF.isScalar() == true


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125856