[Mlir-commits] [mlir] [mlir][scf] Declare ControlFlow as a dependent dialect (PR #216852)

[Matthias Springer]

matthias-springer wrote:

> is deleting MultiBlockExecuteInliner the direction you would want? I am happy to write that patch instead.

I wouldn't delete it.

Whether is should be a canonicalization? Not sure... The pattern removes "control flow structure" from the IR. That's not something I would have expected from an SCF canonicalization pattern. Maybe it should be part of a different pass. But just creating a pass for this one pattern may be overkill. I would just fix the dialect dependency for now.


https://github.com/llvm/llvm-project/pull/216852