[PATCH] D98365: [ValueTypes][RISCV] Add MVT for v1f16.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 10 11:56:07 PST 2021


craig.topper created this revision.
craig.topper added a reviewer: frasercrmck.
Herald added subscribers: vkmr, dexonsmith, evandro, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, simoncook, johnrusso, rbar, asb, hiraditya, dschuff.
craig.topper requested review of this revision.
Herald added subscribers: MaskRay, aheejin.
Herald added a project: LLVM.

RISCV makes all fixed vector MVTs with size less than or equal
to a command line option legal.

This didn't include v1f16 because it was missing but did include v1f32 and v1f64.

One test is affectd where we did test this type, but it is a horizontal
reduction so it is non-sensical. Perhaps we should canonicalize that
away somewhere.

I'm not sure if we should be making v1 types legal, but this will at
least make RISCV consistent across all types.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D98365

Files:
  llvm/include/llvm/CodeGen/ValueTypes.td
  llvm/include/llvm/Support/MachineValueType.h
  llvm/lib/CodeGen/ValueTypes.cpp
  llvm/test/CodeGen/RISCV/rvv/fixed-vectors-reduction-fp.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D98365.329724.patch
Type: text/x-patch
Size: 26873 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210310/6b4e8e5e/attachment.bin>


More information about the llvm-commits mailing list