tree ebf4e311922b2b590a4da7772e033d93f13df297
parent e152f085208636df73d17f25c649c7f0aa4ccfb6
author Alex Zinenko <zinenko@google.com> 1612948310 +0100
committer Copybara-Service <copybara-worker@google.com> 1613725307 -0800

[mlir] avoid exposing mutable DialectRegistry from MLIRContext

MLIRContext allows its users to access directly to the DialectRegistry it
contains. While sometimes useful for registering additional dialects on an
already existing context, this breaks the encapsulation by essentially giving
raw accesses to a part of the context's internal state. Remove this mutable
access and instead provide a method to append a given DialectRegistry to the
one already contained in the context. Also provide a shortcut mechanism to
construct a context from an already existing registry, which seems to be a
common use case in the wild. Keep read-only access to the registry contained in
the context in case it needs to be copied or used for constructing another
context.

With this change, DialectRegistry is no longer concerned with loading the
dialects and deciding whether to invoke delayed interface registration. Loading
is concentrated in the MLIRContext, and the functionality of the registry
better reflects its name.

Depends On D96137

Reviewed By: mehdi_amini

Differential Revision: https://reviews.llvm.org/D96331

GitOrigin-RevId: 2996a8d67553b9d469e01215b49bb1af17ad6d1e