[Mlir-commits] [mlir] [MLIR][NFC] Add allow Insert/extract slice option to pack/unpack op (PR #117340)

[llvmlistbot at llvm.org]

Max191 wrote:

> On a different note, there's `lower{Un}Pack` and there's `DecomposeOuterUnitDimsPackOpPattern`. We should unify these things at some point.

I agree it is good to unify and reduce the overall number of patterns, but just want to note that we should make sure we still have control over how the patterns are applied (that's probably what you were suggesting anyway, but just wanted to add that comment).

https://github.com/llvm/llvm-project/pull/117340