[Mlir-commits] [mlir] 9610a2d - [MLIR] Add memref dialect dependency for affine fusion pass

[Uday Bondhugula]

Author: Vinayaka Bandishti
Date: 2021-05-08T20:12:33+05:30
New Revision: 9610a2d753dbba385e8c2c005e2497e3add99472

URL: https://github.com/llvm/llvm-project/commit/9610a2d753dbba385e8c2c005e2497e3add99472
DIFF: https://github.com/llvm/llvm-project/commit/9610a2d753dbba385e8c2c005e2497e3add99472.diff

LOG: [MLIR] Add memref dialect dependency for affine fusion pass

For `AffineLoopFusion` pass, add `memref` dialect as a dependent
dialect. Since the fusion pass can create `memref::AllocOp`s, the
dialect must be registered in its dependent dialects.

The missing dependency was not discovered until now because the above
said op creation happes only when the input already has
`memref::AllocOp`s in it, and all dialects in the input are
automatically added to the context.

Reviewed By: bondhugula

Differential Revision: https://reviews.llvm.org/D102104

Added: 
    

Modified: 
    mlir/include/mlir/Transforms/Passes.td

Removed: 
    


################################################################################
diff  --git a/mlir/include/mlir/Transforms/Passes.td b/mlir/include/mlir/Transforms/Passes.td
index 39dc3477723c7..d14216b24b0d8 100644
--- a/mlir/include/mlir/Transforms/Passes.td
+++ b/mlir/include/mlir/Transforms/Passes.td
@@ -136,6 +136,7 @@ def AffineLoopFusion : FunctionPass<"affine-loop-fusion"> {
     Option<"maximalFusion", "fusion-maximal", "bool", /*default=*/"false",
            "Enables maximal loop fusion">,
   ];
+  let dependentDialects = ["memref::MemRefDialect"];
 }
 
 def AffinePipelineDataTransfer