[Mlir-commits] [mlir] [mlir][py] partially use mlir_type_subclass for IRTypes.cpp (PR #171143)

[Oleksandr Alex Zinenko]

ftynse wrote:

Just to summarize an offline discussion here for public knowledge: I went with `mlir_type_subclass` because I recall there were problems with exposing `PyConcreteType` to downstreams because of the infamous shared/static library problem due to ODR violations from having multiple copies of the same binding logic coming from different subprojects. If this is resolved, I'm happy to take `PyConcreteType` instead. @makslevental will revive #156575 and we will discuss based on that.

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