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

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

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

> @lebedev.ri Some of the x86 interleaved codegen sees some regressions from this

Yep, so i've seen. This is why we *desperately* *desperately* need 'live' costmodel that would be automatically produced from current codegen.

> are you at all familiar with any of it?

Not particularly. I can only state the obvious: we are probably missing a lot of combines to recover from whatever the changes were to the legalization.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115653