[PATCH] D129351: [RISCV] Add cost model for vector.splice mask operation

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 12 10:08:48 PDT 2022


reames requested changes to this revision.
reames added a comment.
This revision now requires changes to proceed.

I'm a bit confused.  It sounds like the sequence you're describing is inserted during legalization?  If so, why is this not covered by the existing use of legalization cost?

Also, it would help me to understand your intent if you included the generated code for an example in the review description.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129351



More information about the llvm-commits mailing list