[PATCH] D148713: [LegalizeTypes][VP] Widen load/store of fixed length vectors to VP ops

[Luke Lau via Phabricator via llvm-commits]

luke added inline comments.


================
Comment at: llvm/test/CodeGen/RISCV/rvv/fixed-vectors-insert.ll:65
 ; RV64:       # %bb.0:
-; RV64-NEXT:    sd a1, 16(a0)
 ; RV64-NEXT:    ret
----------------
We lose this optimisation because we're no longer scalarizing the vector loads and stores. Can we add a combine for `(store p, (insertelt x, (load p))`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148713