[PATCH] D115653: [DAG]Introduce llvm::processShuffleMasks and use it for shuffles in DAG Type Legalizer.

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

Thanks for the update.

It looks like the Arm code is just getting stuck in various inefficient ARMISD::BUILD_VECTOR missing folds. It was previously going through the useBuildVector=true path so creating BUILDVECTOR earlier. Considering what they are testing, that seems fine.

I have no objections to this, if the X86 folks are OK with this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115653