[Mlir-commits] [mlir] [mlir][canonicalize] Add filter-dialects option (PR #193041)

[Oleksandr Alex Zinenko]

ftynse wrote:

`DialectRegistry` right now is just a mapping between a dialect name and TypeID/constructor. We do use it to communicate pass dependent dialects as well, but adding the notion of "dependency" in the object itself sounds off. Dependent dialects make sense for passes, less for for registries. It also creates a second way for dialects to exist in the registry, but without being able to be constructed.

It may be a matter of naming or structure. For example, we can add the to the mapping with a null constructor and have a general diagnostic when constructing dialects that the constructor is non-null (orthogonal, but it appears possible right now to have that though the discouraged API).

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