[PATCH] D73295: [mlir] [VectorOps] Rewriting of vector.extract/insert_slices to other vector ops
Aart Bik via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 24 14:19:08 PST 2020
aartbik added a comment.
Hmm. I would argue that the tuple discussion makes less sense in the context of this particular rewriting (since it really deals with getting rid of extract_slices/insert_slices using other slice ops). Cleaning up tuple uses is a nice bonus, but not a requirement, since tuples are really a part of the vector dialect to start with, as the leaking example shows. If you want something that blocks any tuple uses, I would argue that we need another pattern population for that, and run that at places where we need to guarantee they are gone (such as the lowering to LLVM, although the "legality" part takes care of it there already).
Other than this comment, I am not sure what you would like to see added in this CL.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D73295/new/
https://reviews.llvm.org/D73295
More information about the llvm-commits
mailing list