Page MenuHomePhabricator

[RISCV] Add scalable-vector integer extension patterns
ClosedPublic

Authored by frasercrmck on Thu, Jan 14, 8:09 AM.

Diff Detail

Event Timeline

frasercrmck created this revision.Thu, Jan 14, 8:09 AM
frasercrmck requested review of this revision.Thu, Jan 14, 8:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptThu, Jan 14, 8:09 AM
  • fix up bad type mangling in tests (e.g. nx4i8 -> nxv4i8)
craig.topper added inline comments.Fri, Jan 15, 12:06 PM
llvm/lib/Target/RISCV/RISCVInstrInfoVSDPatterns.td
317

Should we handle any extend as well? Not sure how to test that with scalable vectors though

  • rebase on main
frasercrmck added inline comments.Mon, Jan 18, 7:18 AM
llvm/lib/Target/RISCV/RISCVInstrInfoVSDPatterns.td
317

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?

add pats for anyext: treat like zext for now

This revision is now accepted and ready to land.Mon, Jan 18, 11:29 PM