[PATCH] D83950: [CodeGen] In narrowExtractedVectorLoad bail out for scalable vectors

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen added a comment.

LGTM, thanks!



================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:19315
   unsigned Index = ExtIdx->getZExtValue();
-  unsigned NumElts = VT.getVectorNumElements();
+  unsigned NumElts = VT.getVectorMinNumElements();
 
----------------
Can you make this ElementCount, to make it clear that it can be scalable?


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

https://reviews.llvm.org/D83950