[PATCH] D95967: [DAGCombiner][SVE] Fix invalid use of getVectorNumElements() in visitSRA.

[David Sherwood via Phabricator via llvm-commits]

david-arm added a comment.

Hi, this patch looks like it has some good fixes, thanks! I just left one comment about adding a test if possible?



================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:8419
       if (VT.isVector())
-        TruncVT = EVT::getVectorVT(Ctx, TruncVT, VT.getVectorNumElements());
+        TruncVT = EVT::getVectorVT(Ctx, TruncVT, VT.getVectorElementCount());
 
----------------
Is it possible to add a test for this too?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95967