[PATCH] D94694: [RISCV] Add scalable-vector integer extension patterns

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoVSDPatterns.td:312
+// 12.3. Vector Integer Extension
+defm "" : VPatExtendSDNode_V<zext, "PseudoVZEXT", "VF2", AllFractionableVF2IntVectors>;
+defm "" : VPatExtendSDNode_V<sext, "PseudoVSEXT", "VF2", AllFractionableVF2IntVectors>;
----------------
craig.topper wrote:
> Should we handle any extend as well? Not sure how to test that with scalable vectors though
We probably should do any extend, yes. I've been looking around the codebase and couldn't really find an "in" to be able to test it in vanilla llvm. I don't think that should stop us, though. Do we have a preference for mapping it to sext or zext? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94694