[Mlir-commits] [mlir] [mlir][memref] Use array notation instead of permutation map for memref.transpose (PR #67880)

Felix Schneider llvmlistbot at llvm.org
Mon Oct 2 02:41:09 PDT 2023


ubfx wrote:

> 1. Why would the array solve that?

It doesn't remove the ambiguity issue completely, but the argument would be that if neither notation is 100% unambiguous, at least one of them is consistent with the other `transpose` Ops within the project. 

But I guess the consistency point is better addressed by this:
> Even better, if permutations are such a common case, we could introduce a `PermutationAttr` with unambiguous API backed by a dense storage.



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


More information about the Mlir-commits mailing list