[PATCH] D107597: [DAG] Fold concat_vectors(concat_vectors(x,y),concat_vectors(a,b)) -> concat_vectors(x,y,a,b)

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck added a comment.

The RISC-V changes look good to me.



================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:19869
+  // TODO: Add scalable vector type support.
+  if (!VT.isFixedLengthVector())
+      return SDValue();
----------------
What would it take to add scalable vector support? I don't think I see anything here that relies on fixed-vector knowledge. If it's lack of testing we may be able to rustle up some test cases with extra wide scalable-vectors which are split during legalization?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107597