[Mlir-commits] [mlir] [mlir][transform] Handle multiple library preloading passes. (PR #69705)

Ingo Müller llvmlistbot at llvm.org
Tue Oct 24 03:29:23 PDT 2023


================
@@ -78,20 +78,19 @@ def Transform_Dialect : Dialect {
     using ExtensionTypePrintingHook =
         std::function<void(::mlir::Type, ::mlir::AsmPrinter &)>;
 
-    /// Appends the given module as a transform symbol library available to
-    /// all dialect users.
-    void registerLibraryModule(::mlir::OwningOpRef<::mlir::ModuleOp> &&
-                                library) {
-      libraryModules.push_back(std::move(library));
-    }
-
-    /// Returns a range of registered library modules.
-    auto getLibraryModules() const {
-      return ::llvm::map_range(
-          libraryModules,
-          [](const ::mlir::OwningOpRef<::mlir::ModuleOp> &library) {
-        return library.get();
-      });
+    /// Loads the given module into the transform symbol library module.
----------------
ingomueller-net wrote:

Yes, and also unnecessarily public. Fixed, thanks!

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


More information about the Mlir-commits mailing list