This is an archive of the discontinued LLVM Phabricator instance.

[MLIR] Add memref dialect dependency for affine fusion pass
ClosedPublic

Authored by vinayaka-polymage on May 8 2021, 6:22 AM.

Details

Summary

For AffineLoopFusion pass, add memref dialect as a dependent
dialect. Since the fusion pass can create memref::AllocOps, 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::AllocOps in it, and all dialects in the input are
automatically added to the context.

Diff Detail

Event Timeline

vinayaka-polymage requested review of this revision.May 8 2021, 6:22 AM

Better commit summary and message

vinayaka-polymage retitled this revision from [MLIR] Fusion to have `memref` as a dependent dialect to [MLIR] Add memref dialect dependency for affine fusion pass.
vinayaka-polymage edited the summary of this revision. (Show Details)

Adding why the issue was not discovered till now

bondhugula accepted this revision.EditedMay 8 2021, 7:41 AM

Thanks! Interesting that this wasn't discovered this long! All test cases that required generation of an AllocOp in the fusion pass had to be those that were locally defined - so they already had an alloc op.

This revision is now accepted and ready to land.May 8 2021, 7:41 AM
This revision was landed with ongoing or failed builds.May 8 2021, 7:46 AM
This revision was automatically updated to reflect the committed changes.