[Mlir-commits] [mlir] [mlir][python] remove mixins (PR #68853)

Maksim Levental llvmlistbot at llvm.org
Wed Oct 18 17:50:29 PDT 2023


makslevental wrote:

> it seems it is mostly just the "constructor" that is modified and potentially some extra class members, but don't know if there is mechanism to make that easy while ensuring SOT is ODS still. 

One thing to consider, if this is a concern, is to drop registering the mixins as the default `op.opview`. Then they could break in an of themselves (which would be embarrassing) but that breakage would be localized a single instantiation.

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


More information about the Mlir-commits mailing list