[PATCH] D76637: [mlir][Linalg] Allow tiling of batch dimension for convolution ops with padding.

Uday Bondhugula via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 23 12:33:23 PDT 2020


bondhugula added inline comments.


================
Comment at: mlir/lib/Dialect/Linalg/Transforms/Tiling.cpp:444
 
+  if (auto convOp = dyn_cast<linalg::ConvOp>(op.getOperation())) {
+    // For conv op only support tiling along batch dimension (which is the first
----------------
bondhugula wrote:
> You don't need the getOperation(). 
Sorry, you do need it here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76637





More information about the llvm-commits mailing list