[Mlir-commits] [mlir] [mlir][linalg] Restrict linalg.pack to not have extra padding sizes. (PR #149624)

[Adam Siemieniuk]

https://github.com/adam-smnk commented:

I like the overall concept.
Creating extra tiles filled only with padding value does not seem critical to the packing itself. The same behavior should be achievable with a separate, follow-up pad. So, the current pad semantics increase ops' complexity with no clear payoff.

The PR description could use clearer wording. `extra padding` is too ambiguous as packing requires some amount of padding.
Once new desired semantics are fleshed out, the `linalg.pack` op description should be updated too.

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