[PATCH] D104156: [DAGCombine][X86][ARM] EXTRACT_SUBVECTOR(VECTOR_SHUFFLE(?,?,Mask)) -> VECTOR_SHUFFLE(EXTRACT_SUBVECTOR(?, ?), EXTRACT_SUBVECTOR(?, ?), Mask')

Wed Dec 8 12:03:02 PST 2021

lebedev.ri added inline comments.

================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:20794
+                   .drop_front(FirstExtractedEltIdx)
+                   .take_front(NumEltsExtracted)) {
+    assert((M >= -1) && (M < (2 * WideNumElts)) &&
----------------
RKSimon wrote:
> RKSimon wrote:
> > WideShuffleVector->getMask().slice() ?
> slice() ?
Ugh, sorry, that change got lost when i rolled back to previous revision without double-checking...

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104156