[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