[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