[PATCH] D76460: [mlir] [VectorOps] Introduce vector.transpose on 2-D vectors

Aart Bik via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 19 18:39:58 PDT 2020


aartbik added a comment.

In D76460#1932449 <https://reviews.llvm.org/D76460#1932449>, @andydavis1 wrote:

> Thanks Aart. How hard would this be to generalize this op to make it N-D transpose op, instead of limit to 2D cases? For example, I may want to transpose the inner two dimensions of a N-D vector


Not very hard for the reference implementation (so the direct lowering into a bunch of extra/insert operations), but a lot harder for efficient implementations.
Would you rather that I lift the restriction on 2-D right now already and provide a reference lowering for all ranks and sizes?
(happy to do that)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76460





More information about the llvm-commits mailing list