[PATCH] D75831: [mlir][Linalg] Revisit 0-D abstraction
Mahesh Ravishankar via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 10 00:53:02 PDT 2020
mravishankar requested changes to this revision.
mravishankar added a comment.
This revision now requires changes to proceed.
I looked at this change a little bit deeper, and I am not sure this works. This seems to have an affine_map for scalars being () -> (). This seems odd. Isn't it better to have a scalar access be (i,j) -> (), or something like that.
That needs change to the AffineMap constructor above.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75831/new/
https://reviews.llvm.org/D75831
More information about the llvm-commits
mailing list