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

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 14 06:21:53 PDT 2022


ABataev added a comment.

In D115653#3379258 <https://reviews.llvm.org/D115653#3379258>, @RKSimon wrote:

> What about if we split the addition of  VectorUtils processShuffleMasks off from the DAG legalization code - that way we can get D100486 <https://reviews.llvm.org/D100486> moving again and address the DAG regressions separately?

Anyway it is good to investigate regressions before that, may help improve final code. I'll try to make it ASAP. Better doublecheck rather than doing reverts again and again.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115653



More information about the llvm-commits mailing list