Constant-0 dim expr values should be avoided for linalg as it can prevent
fusion. This includes adding support for rank-0 reshapes.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Nice!
mlir/lib/Conversion/TosaToLinalg/TosaToLinalg.cpp | ||
---|---|---|
448 | I dont think you need template dyn_cast<...> here. That is only needed if the method is templated. | |
501 | You might as well use a linalg.tensor_reshape, but this is fine too. FYI, for tosa::ReshapeOp you can use the utility method here https://reviews.llvm.org/D101343 which should be able to give you the reassociation maps that you need for lowring the reshape to linalg. |
mlir/lib/Conversion/TosaToLinalg/TosaToLinalg.cpp | ||
---|---|---|
501 | I'll do a pass back through once your review lands. There are a few areas I can clean up this way. |
I dont think you need template dyn_cast<...> here. That is only needed if the method is templated.