[Mlir-commits] [mlir] [mlir] Declare promised interfaces for all dialects (PR #78368)

[Mehdi Amini]

joker-eph wrote:

> It couples Tensor to Bufferization and to Transform dialect. While technically this can be made a header-only dependency (right now it's not),

The promise declarations are meant to be header only, why is it not right now?


> After this change you're forced to load all the bufferization interfaces for all the ops, which breaks this use case.

The promise only forces you to load what you use right now: are you saying that you have ops in your IR for which you want to run the pass but you want the `dyn_cast<BufferizationOpIntertace>(op)` to intentionally fail?
I don't quite see that as something that can (and more importantly should!) be supported. This is not the right mechanism, you need to look into the buffization pass itself, and/or inject an interface implementation that behaves as you'd expect for these operations.





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