[Mlir-commits] [mlir] [mlir] load dialects for non-namespaced attrs (PR #94838)
Mehdi Amini
llvmlistbot at llvm.org
Wed Jun 12 16:30:14 PDT 2024
================
@@ -144,6 +144,10 @@ TranslateFromMLIRRegistration::TranslateFromMLIRRegistration(
DialectRegistry registry;
dialectRegistration(registry);
context->appendDialectRegistry(registry);
+ // Ensure all registered dialects are loaded
+ for (const auto &dialectName : registry.getDialectNames()) {
+ context->getOrLoadDialect(dialectName);
+ }
----------------
joker-eph wrote:
That does not seem like the right fix indeed, the parser should load on demand.
I need to reproduce to figure out how we get to crash.
https://github.com/llvm/llvm-project/pull/94838
More information about the Mlir-commits
mailing list