[PATCH] D119529: [BasicTTI] Set scalarization cost of getCommonMaskedMemoryOpCost to Invalid.

Jim Lin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 14 22:22:51 PST 2022


Jim added a comment.

In D119529#3313680 <https://reviews.llvm.org/D119529#3313680>, @sdesmalen wrote:

> Can you add a test for this change?

The crash only triggered target is given (specify -mtriple=riscv64) and BasicTTIImplBase::getCommonMaskedMemoryOpCost would be called.
If no target given, NoTTIImpl is used as TTI, and TargetTransformInfoImpl.h::getMemoryOpCost simply returns cost 1 without any crash.

I don't have plan to implement hook getMaskedMemoryOpCost for RISCV. 
Do you have any suggest for this kind situation?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119529



More information about the llvm-commits mailing list