This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Add scalable-vector integer extension patterns
ClosedPublic

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

Diff Detail

Event Timeline

frasercrmck created this revision.Jan 14 2021, 8:09 AM
frasercrmck requested review of this revision.Jan 14 2021, 8:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 14 2021, 8:09 AM
  • fix up bad type mangling in tests (e.g. nx4i8 -> nxv4i8)
craig.topper added inline comments.Jan 15 2021, 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.Jan 18 2021, 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.Jan 18 2021, 11:29 PM