[Mlir-commits] [mlir] [MLIR][Python] Impl XOpInterface(s) from Python, with X=Transform and X=MemoryEffects (PR #176920)

[llvmlistbot at llvm.org]

================
@@ -206,8 +206,7 @@ PyGlobals::lookupOperationClass(llvm::StringRef operationName) {
   // Make sure dialect module is loaded.
   auto split = operationName.split('.');
   llvm::StringRef dialectNamespace = split.first;
-  if (!loadDialectModule(dialectNamespace))
-    return std::nullopt;
+  loadDialectModule(dialectNamespace);
----------------
PragmaTwice wrote:

ahh this is because `loadDialectModule` doesn't work for python-defined dialects. i'm checking if there's a way to support python-defined dialects, but currently I think we should search for operation classes even `loadDialectModule` fails. because users can import some module manually.

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