[Mlir-commits] [mlir] [mlir] Add optional layout attribute to VectorType (PR #71916)

[Renato Golin]

https://github.com/rengolin commented:

The code looks ok to me, the abstraction level to keep it implementation defined, too. Should be fine for the hands-off approach we're having now.

I wonder if we want to have some implementation upstream, though. Sure, this would be for another PR, but how many upstream passes are we going to disable because they don't understand the downstream layout encoding?

I'll leave for others to finally approve the PR, to give them time to review.

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