[PATCH] D147713: [RISCV] Combine concat_vectors of loads into strided loads

[Luke Lau via Phabricator via llvm-commits]

luke marked 6 inline comments as done.
luke added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:11441
+    else if (BaseLdVT.isFloatingPoint())
+      WideScalarVT = MVT::getFloatingPointVT(WideScalarBitWidth);
+    else
----------------
craig.topper wrote:
> Wondering if we should always do integer in case f64 vector isn't supported, but i64 is?
Good point, I couldn't recreate a test case for this though because we always check that `VT` is legal first. I also tried removing the legal type check, but then we get an assertion when calling `convertFromScalableVector` with an illegal type.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147713