[PATCH] D121677: [RISCV] Return Invalid cost in getGatherScatterOpCost instead of crashing for scalable vectors
Liao Chunyu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 30 23:26:10 PDT 2022
liaolucy added a comment.
Herald added subscribers: sunshaoce, StephenFan.
In D121677#3386240 <https://reviews.llvm.org/D121677#3386240>, @frasercrmck wrote:
> I actually hit this same issue the day you posted this, which is fun. But I fear this is quite a lot of work to get watertight. I've left some comments showing where we'd still crash. I've also seen us crash on `getIntrinsicInstrCost` when given, e.g., `llvm.cttz.nxv1i8`.
Can adding a cost model for llvm.cttz.nxv1i8 solve this problem? I would like to try.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121677/new/
https://reviews.llvm.org/D121677
More information about the llvm-commits
mailing list