[PATCH] D159230: [RISCV] Handle EltType > XLEN case in VMV_V_X_VL to VMV_S_X_VL fold

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 30 14:31:56 PDT 2023


reames created this revision.
reames added reviewers: craig.topper, asb, luke.
Herald added subscribers: jobnoorman, sunshaoce, VincentWu, vkmr, frasercrmck, 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, bollu, simoncook, johnrusso, rbar, hiraditya, arichardson, mcrosier.
Herald added a project: All.
reames requested review of this revision.
Herald added subscribers: wangpc, eopXD, MaskRay.
Herald added a project: LLVM.

I'd guarded this case in D158874 <https://reviews.llvm.org/D158874> to avoid regressions, and decided to go investigate what was going on.  The solution turns out to be a generic splat matching extension to handle INSERT_SUBVECTOR.  In theory, we could see these from other sources as well, but for some reason we only seem to see the i64 extract on rv32 case in practice.  Not sure why that is to be honest.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D159230

Files:
  llvm/lib/Target/RISCV/RISCVISelDAGToDAG.cpp
  llvm/lib/Target/RISCV/RISCVISelLowering.cpp
  llvm/test/CodeGen/RISCV/rvv/fixed-vectors-vrol.ll
  llvm/test/CodeGen/RISCV/rvv/fixed-vectors-vror.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D159230.554836.patch
Type: text/x-patch
Size: 6859 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230830/1492a0cd/attachment.bin>


More information about the llvm-commits mailing list