[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