Revert "Implement Pass and Dialect plugins for mlir-opt"
authorJacques Pienaar <jpienaar@google.com>
Thu, 6 Apr 2023 16:41:22 +0000 (09:41 -0700)
committerJacques Pienaar <jpienaar@google.com>
Thu, 6 Apr 2023 16:41:51 +0000 (09:41 -0700)
Breaks bot.

This reverts commit d4c873b044aeebaa43e6989fb1f9606530ec60cf.

21 files changed:
mlir/examples/standalone/CMakeLists.txt
mlir/examples/standalone/include/Standalone/CMakeLists.txt
mlir/examples/standalone/include/Standalone/StandalonePasses.h [deleted file]
mlir/examples/standalone/include/Standalone/StandalonePasses.td [deleted file]
mlir/examples/standalone/lib/Standalone/CMakeLists.txt
mlir/examples/standalone/lib/Standalone/StandalonePasses.cpp [deleted file]
mlir/examples/standalone/standalone-opt/standalone-opt.cpp
mlir/examples/standalone/standalone-plugin/CMakeLists.txt [deleted file]
mlir/examples/standalone/standalone-plugin/standalone-plugin.cpp [deleted file]
mlir/examples/standalone/test/Standalone/standalone-pass-plugin.mlir [deleted file]
mlir/examples/standalone/test/Standalone/standalone-plugin.mlir [deleted file]
mlir/examples/standalone/test/lit.cfg.py
mlir/include/mlir/IR/DialectPlugin.h [deleted file]
mlir/include/mlir/Pass/PassPlugin.h [deleted file]
mlir/include/mlir/Tools/mlir-opt/MlirOptMain.h
mlir/lib/IR/CMakeLists.txt
mlir/lib/IR/DialectPlugin.cpp [deleted file]
mlir/lib/Pass/CMakeLists.txt
mlir/lib/Pass/PassPlugin.cpp [deleted file]
mlir/lib/Tools/mlir-opt/MlirOptMain.cpp
mlir/tools/mlir-opt/CMakeLists.txt

index 65461c0..d36a6ba 100644 (file)
@@ -52,5 +52,4 @@ if(MLIR_ENABLE_BINDINGS_PYTHON)
 endif()
 add_subdirectory(test)
 add_subdirectory(standalone-opt)
-add_subdirectory(standalone-plugin)
 add_subdirectory(standalone-translate)
index 975a4ff..8acf640 100644 (file)
@@ -1,7 +1,3 @@
 add_mlir_dialect(StandaloneOps standalone)
 add_mlir_doc(StandaloneDialect StandaloneDialect Standalone/ -gen-dialect-doc)
 add_mlir_doc(StandaloneOps StandaloneOps Standalone/ -gen-op-doc)
-
-set(LLVM_TARGET_DEFINITIONS StandalonePasses.td)
-mlir_tablegen(StandalonePasses.h.inc --gen-pass-decls)
-add_public_tablegen_target(MLIRStandalonePassesIncGen)
diff --git a/mlir/examples/standalone/include/Standalone/StandalonePasses.h b/mlir/examples/standalone/include/Standalone/StandalonePasses.h
deleted file mode 100644 (file)
index 75546d6..0000000
+++ /dev/null
@@ -1,26 +0,0 @@
-//===- StandalonePasses.h - Standalone passes  ------------------*- C++ -*-===//
-//
-// This file is licensed under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
-#ifndef STANDALONE_STANDALONEPASSES_H
-#define STANDALONE_STANDALONEPASSES_H
-
-#include "Standalone/StandaloneDialect.h"
-#include "Standalone/StandaloneOps.h"
-#include "mlir/Pass/Pass.h"
-#include <memory>
-
-namespace mlir {
-namespace standalone {
-#define GEN_PASS_DECL
-#include "Standalone/StandalonePasses.h.inc"
-
-#define GEN_PASS_REGISTRATION
-#include "Standalone/StandalonePasses.h.inc"
-} // namespace standalone
-} // namespace mlir
-
-#endif
diff --git a/mlir/examples/standalone/include/Standalone/StandalonePasses.td b/mlir/examples/standalone/include/Standalone/StandalonePasses.td
deleted file mode 100644 (file)
index 4cb2be0..0000000
+++ /dev/null
@@ -1,30 +0,0 @@
-//===- StandalonePsss.td - Standalone dialect passes -------*- tablegen -*-===//
-//
-// This file is licensed under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
-
-#ifndef STANDALONE_PASS
-#define STANDALONE_PASS
-
-include "mlir/Pass/PassBase.td"
-
-def StandaloneSwitchBarFoo: Pass<"standalone-switch-bar-foo", "::mlir::ModuleOp"> {
-  let summary = "Switches the name of a FuncOp named `bar` to `foo` and folds.";
-  let description = [{
-    Switches the name of a FuncOp named `bar` to `foo` and folds.
-    ```
-    func.func @bar() {
-      return
-    }
-    // Gets transformed to:
-    func.func @foo() {
-      return
-    }
-    ```
-  }];
-}
-
-#endif // STANDALONE_PASS
index 0e2d043..599c1c5 100644 (file)
@@ -2,17 +2,14 @@ add_mlir_dialect_library(MLIRStandalone
         StandaloneTypes.cpp
         StandaloneDialect.cpp
         StandaloneOps.cpp
-        StandalonePasses.cpp
 
         ADDITIONAL_HEADER_DIRS
         ${PROJECT_SOURCE_DIR}/include/Standalone
 
         DEPENDS
         MLIRStandaloneOpsIncGen
-        MLIRStandalonePassesIncGen
 
        LINK_LIBS PUBLIC
        MLIRIR
         MLIRInferTypeOpInterface
-        MLIRFuncDialect
        )
diff --git a/mlir/examples/standalone/lib/Standalone/StandalonePasses.cpp b/mlir/examples/standalone/lib/Standalone/StandalonePasses.cpp
deleted file mode 100644 (file)
index 6af45c9..0000000
+++ /dev/null
@@ -1,48 +0,0 @@
-//===- StandalonePasses.cpp - Standalone passes -----------------*- C++ -*-===//
-//
-// This file is licensed under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
-#include "mlir/Dialect/Func/IR/FuncOps.h"
-#include "mlir/IR/BuiltinOps.h"
-#include "mlir/IR/PatternMatch.h"
-#include "mlir/Pass/Pass.h"
-#include "mlir/Transforms/GreedyPatternRewriteDriver.h"
-
-#include "Standalone/StandalonePasses.h"
-
-namespace mlir::standalone {
-#define GEN_PASS_DEF_STANDALONESWITCHBARFOO
-#include "Standalone/StandalonePasses.h.inc"
-
-namespace {
-class StandaloneSwitchBarFooRewriter : public OpRewritePattern<func::FuncOp> {
-public:
-  using OpRewritePattern<func::FuncOp>::OpRewritePattern;
-  LogicalResult matchAndRewrite(func::FuncOp op,
-                                PatternRewriter &rewriter) const final {
-    if (op.getSymName() == "bar") {
-      rewriter.updateRootInPlace(op, [&op]() { op.setSymName("foo"); });
-      return success();
-    }
-    return failure();
-  }
-};
-
-class StandaloneSwitchBarFoo
-    : public impl::StandaloneSwitchBarFooBase<StandaloneSwitchBarFoo> {
-public:
-  using impl::StandaloneSwitchBarFooBase<
-      StandaloneSwitchBarFoo>::StandaloneSwitchBarFooBase;
-  void runOnOperation() final {
-    RewritePatternSet patterns(&getContext());
-    patterns.add<StandaloneSwitchBarFooRewriter>(&getContext());
-    FrozenRewritePatternSet patternSet(std::move(patterns));
-    if (failed(applyPatternsAndFoldGreedily(getOperation(), patternSet)))
-      signalPassFailure();
-  }
-};
-} // namespace
-} // namespace mlir::standalone
index e75db35..4cf7e15 100644 (file)
 #include "llvm/Support/ToolOutputFile.h"
 
 #include "Standalone/StandaloneDialect.h"
-#include "Standalone/StandalonePasses.h"
 
 int main(int argc, char **argv) {
   mlir::registerAllPasses();
-  mlir::standalone::registerPasses();
   // TODO: Register standalone passes here.
 
   mlir::DialectRegistry registry;
diff --git a/mlir/examples/standalone/standalone-plugin/CMakeLists.txt b/mlir/examples/standalone/standalone-plugin/CMakeLists.txt
deleted file mode 100644 (file)
index 1b8ca38..0000000
+++ /dev/null
@@ -1,21 +0,0 @@
-get_property(dialect_libs GLOBAL PROPERTY MLIR_DIALECT_LIBS)
-get_property(conversion_libs GLOBAL PROPERTY MLIR_CONVERSION_LIBS)
-set(LIBS
-        MLIRIR
-        MLIRPass
-        MLIRStandalone
-        MLIRTransformUtils
-        )
-
-add_mlir_dialect_library(StandalonePlugin
-        SHARED
-        standalone-plugin.cpp
-
-        DEPENDS
-        MLIRStandalone
-        )
-
-llvm_update_compile_flags(StandalonePlugin)
-target_link_libraries(StandalonePlugin PRIVATE ${LIBS})
-
-mlir_check_all_link_libraries(StandalonePlugin)
diff --git a/mlir/examples/standalone/standalone-plugin/standalone-plugin.cpp b/mlir/examples/standalone/standalone-plugin/standalone-plugin.cpp
deleted file mode 100644 (file)
index e6b2c07..0000000
+++ /dev/null
@@ -1,39 +0,0 @@
-//===- standalone-plugin.cpp ------------------------------------*- C++ -*-===//
-//
-// This file is licensed under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
-
-#include "mlir/Dialect/Arith/IR/Arith.h"
-#include "mlir/IR/Dialect.h"
-#include "mlir/IR/DialectPlugin.h"
-#include "mlir/IR/MLIRContext.h"
-#include "mlir/InitAllDialects.h"
-#include "mlir/InitAllPasses.h"
-#include "mlir/Pass/Pass.h"
-
-#include "Standalone/StandaloneDialect.h"
-#include "Standalone/StandalonePasses.h"
-
-using namespace mlir;
-
-/// Dialect plugin registration mechanism.
-/// Observe that it also allows to register passes.
-/// Necessary symbol to register the dialect plugin.
-extern "C" LLVM_ATTRIBUTE_WEAK DialectPluginLibraryInfo
-mlirGetDialectPluginInfo() {
-  return {MLIR_PLUGIN_API_VERSION, "Standalone", LLVM_VERSION_STRING,
-          [](DialectRegistry *registry) {
-            registry->insert<mlir::standalone::StandaloneDialect>();
-            mlir::standalone::registerPasses();
-          }};
-}
-
-/// Pass plugin registration mechanism.
-/// Necessary symbol to register the pass plugin.
-extern "C" LLVM_ATTRIBUTE_WEAK PassPluginLibraryInfo mlirGetPassPluginInfo() {
-  return {MLIR_PLUGIN_API_VERSION, "StandalonePasses", LLVM_VERSION_STRING,
-          []() { mlir::standalone::registerPasses(); }};
-}
diff --git a/mlir/examples/standalone/test/Standalone/standalone-pass-plugin.mlir b/mlir/examples/standalone/test/Standalone/standalone-pass-plugin.mlir
deleted file mode 100644 (file)
index 5af4b3d..0000000
+++ /dev/null
@@ -1,13 +0,0 @@
-// RUN: mlir-opt %s --load-pass-plugin=%standalone_libs/libStandalonePlugin.so --pass-pipeline="builtin.module(standalone-switch-bar-foo)" | FileCheck %s
-
-module {
-  // CHECK-LABEL: func @foo()
-  func.func @bar() {
-    return
-  }
-
-  // CHECK-LABEL: func @abar()
-  func.func @abar() {
-    return
-  }
-}
diff --git a/mlir/examples/standalone/test/Standalone/standalone-plugin.mlir b/mlir/examples/standalone/test/Standalone/standalone-plugin.mlir
deleted file mode 100644 (file)
index 3f935db..0000000
+++ /dev/null
@@ -1,13 +0,0 @@
-// RUN: mlir-opt %s --load-dialect-plugin=%standalone_libs/libStandalonePlugin.so --pass-pipeline="builtin.module(standalone-switch-bar-foo)" | FileCheck %s
-
-module {
-  // CHECK-LABEL: func @foo()
-  func.func @bar() {
-    return
-  }
-
-  // CHECK-LABEL: func @standalone_types(%arg0: !standalone.custom<"10">)
-  func.func @standalone_types(%arg0: !standalone.custom<"10">) {
-    return
-  }
-}
index 3e4ceee..a6a3d24 100644 (file)
@@ -44,16 +44,12 @@ config.excludes = ['Inputs', 'Examples', 'CMakeLists.txt', 'README.txt', 'LICENS
 # test_exec_root: The root path where tests should be run.
 config.test_exec_root = os.path.join(config.standalone_obj_root, 'test')
 config.standalone_tools_dir = os.path.join(config.standalone_obj_root, 'bin')
-config.standalone_libs_dir = os.path.join(config.standalone_obj_root, 'lib')
-
-config.substitutions.append(('%standalone_libs', config.standalone_libs_dir))
 
 # Tweak the PATH to include the tools dir.
 llvm_config.with_environment('PATH', config.llvm_tools_dir, append_path=True)
 
 tool_dirs = [config.standalone_tools_dir, config.llvm_tools_dir]
 tools = [
-    'mlir-opt',
     'standalone-capi-test',
     'standalone-opt',
     'standalone-translate',
diff --git a/mlir/include/mlir/IR/DialectPlugin.h b/mlir/include/mlir/IR/DialectPlugin.h
deleted file mode 100644 (file)
index df30faa..0000000
+++ /dev/null
@@ -1,106 +0,0 @@
-//===- mlir/IR/DialectPlugin.h - Public Plugin API -----------------------===//
-//
-// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
-//
-// This defines the public entry point for dialect plugins.
-//
-//===----------------------------------------------------------------------===//
-
-#ifndef MLIR_IR_DIALECTPLUGIN_H
-#define MLIR_IR_DIALECTPLUGIN_H
-
-#include "mlir/IR/DialectRegistry.h"
-#include "mlir/Pass/PassPlugin.h"
-#include "llvm/ADT/StringRef.h"
-#include "llvm/Support/Compiler.h"
-#include "llvm/Support/DynamicLibrary.h"
-#include "llvm/Support/Error.h"
-#include <cstdint>
-#include <string>
-
-namespace mlir {
-extern "C" {
-/// Information about the plugin required to load its dialects & passes
-///
-/// This struct defines the core interface for dialect plugins and is supposed
-/// to be filled out by plugin implementors. MLIR-side users of a plugin are
-/// expected to use the \c DialectPlugin class below to interface with it.
-struct DialectPluginLibraryInfo {
-  /// The API version understood by this plugin, usually
-  /// \c MLIR_PLUGIN_API_VERSION
-  uint32_t apiVersion;
-  /// A meaningful name of the plugin.
-  const char *pluginName;
-  /// The version of the plugin.
-  const char *pluginVersion;
-
-  /// The callback for registering dialect plugin with a \c DialectRegistry
-  /// instance
-  void (*registerDialectRegistryCallbacks)(DialectRegistry *);
-};
-}
-
-/// A loaded dialect plugin.
-///
-/// An instance of this class wraps a loaded dialect plugin and gives access to
-/// its interface defined by the \c DialectPluginLibraryInfo it exposes.
-class DialectPlugin {
-public:
-  /// Attempts to load a dialect plugin from a given file.
-  ///
-  /// \returns Returns an error if either the library cannot be found or loaded,
-  /// there is no public entry point, or the plugin implements the wrong API
-  /// version.
-  static llvm::Expected<DialectPlugin> load(const std::string &filename);
-
-  /// Get the filename of the loaded plugin.
-  StringRef getFilename() const { return filename; }
-
-  /// Get the plugin name
-  StringRef getPluginName() const { return info.pluginName; }
-
-  /// Get the plugin version
-  StringRef getPluginVersion() const { return info.pluginVersion; }
-
-  /// Get the plugin API version
-  uint32_t getAPIVersion() const { return info.apiVersion; }
-
-  /// Invoke the DialectRegistry callback registration
-  void
-  registerDialectRegistryCallbacks(DialectRegistry &dialectRegistry) const {
-    info.registerDialectRegistryCallbacks(&dialectRegistry);
-  }
-
-private:
-  DialectPlugin(const std::string &filename,
-                const llvm::sys::DynamicLibrary &library)
-      : filename(filename), library(library), info() {}
-
-  std::string filename;
-  llvm::sys::DynamicLibrary library;
-  DialectPluginLibraryInfo info;
-};
-} // namespace mlir
-
-/// The public entry point for a dialect plugin.
-///
-/// When a plugin is loaded by the driver, it will call this entry point to
-/// obtain information about this plugin and about how to register its dialects.
-/// This function needs to be implemented by the plugin, see the example below:
-///
-/// ```
-/// extern "C" ::mlir::DialectPluginLibraryInfo LLVM_ATTRIBUTE_WEAK
-/// mlirGetDialectPluginInfo() {
-///   return {
-///     MLIR_PLUGIN_API_VERSION, "MyPlugin", "v0.1", [](DialectRegistry) { ... }
-///   };
-/// }
-/// ```
-extern "C" ::mlir::DialectPluginLibraryInfo LLVM_ATTRIBUTE_WEAK
-mlirGetDialectPluginInfo();
-
-#endif /* MLIR_IR_DIALECTPLUGIN_H */
diff --git a/mlir/include/mlir/Pass/PassPlugin.h b/mlir/include/mlir/Pass/PassPlugin.h
deleted file mode 100644 (file)
index a0a0114..0000000
+++ /dev/null
@@ -1,112 +0,0 @@
-//===- mlir/Pass/PassPlugin.h - Public Plugin API -----------------------===//
-//
-// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
-//
-// This defines the public entry point for pass plugins.
-//
-//===----------------------------------------------------------------------===//
-
-#ifndef MLIR_PASSES_PASSPLUGIN_H
-#define MLIR_PASSES_PASSPLUGIN_H
-
-#include "mlir/Support/LLVM.h"
-#include "llvm/ADT/StringRef.h"
-#include "llvm/Support/Compiler.h"
-#include "llvm/Support/DynamicLibrary.h"
-#include "llvm/Support/Error.h"
-#include <cstdint>
-#include <string>
-
-namespace mlir {
-/// \macro MLIR_PLUGIN_API_VERSION
-/// Identifies the API version understood by this plugin.
-///
-/// When a plugin is loaded, the driver will check it's supported plugin version
-/// against that of the plugin. A mismatch is an error. The supported version
-/// will be incremented for ABI-breaking changes to the \c PassPluginLibraryInfo
-/// struct, i.e. when callbacks are added, removed, or reordered.
-#define MLIR_PLUGIN_API_VERSION 1
-
-extern "C" {
-/// Information about the plugin required to load its passes
-///
-/// This struct defines the core interface for pass plugins and is supposed to
-/// be filled out by plugin implementors. LLVM-side users of a plugin are
-/// expected to use the \c PassPlugin class below to interface with it.
-struct PassPluginLibraryInfo {
-  /// The API version understood by this plugin, usually \c
-  /// MLIR_PLUGIN_API_VERSION
-  uint32_t apiVersion;
-  /// A meaningful name of the plugin.
-  const char *pluginName;
-  /// The version of the plugin.
-  const char *pluginVersion;
-
-  /// The callback for registering plugin passes.
-  void (*registerPassRegistryCallbacks)();
-};
-}
-
-/// A loaded pass plugin.
-///
-/// An instance of this class wraps a loaded pass plugin and gives access to
-/// its interface defined by the \c PassPluginLibraryInfo it exposes.
-class PassPlugin {
-public:
-  /// Attempts to load a pass plugin from a given file.
-  ///
-  /// \returns Returns an error if either the library cannot be found or loaded,
-  /// there is no public entry point, or the plugin implements the wrong API
-  /// version.
-  static llvm::Expected<PassPlugin> load(const std::string &filename);
-
-  /// Get the filename of the loaded plugin.
-  StringRef getFilename() const { return filename; }
-
-  /// Get the plugin name
-  StringRef getPluginName() const { return info.pluginName; }
-
-  /// Get the plugin version
-  StringRef getPluginVersion() const { return info.pluginVersion; }
-
-  /// Get the plugin API version
-  uint32_t getAPIVersion() const { return info.apiVersion; }
-
-  /// Invoke the PassRegistry callback registration
-  void registerPassRegistryCallbacks() const {
-    info.registerPassRegistryCallbacks();
-  }
-
-private:
-  PassPlugin(const std::string &filename,
-             const llvm::sys::DynamicLibrary &library)
-      : filename(filename), library(library), info() {}
-
-  std::string filename;
-  llvm::sys::DynamicLibrary library;
-  PassPluginLibraryInfo info;
-};
-} // namespace mlir
-
-/// The public entry point for a pass plugin.
-///
-/// When a plugin is loaded by the driver, it will call this entry point to
-/// obtain information about this plugin and about how to register its passes.
-/// This function needs to be implemented by the plugin, see the example below:
-///
-/// ```
-/// extern "C" ::mlir::PassPluginLibraryInfo LLVM_ATTRIBUTE_WEAK
-/// mlirGetPassPluginInfo() {
-///   return {
-///     MLIR_PLUGIN_API_VERSION, "MyPlugin", "v0.1", []() { ... }
-///   };
-/// }
-/// ```
-extern "C" ::mlir::PassPluginLibraryInfo LLVM_ATTRIBUTE_WEAK
-mlirGetPassPluginInfo();
-
-#endif /* MLIR_PASS_PASSPLUGIN_H */
index f54c29c..bcefc4f 100644 (file)
@@ -35,12 +35,12 @@ class PassManager;
 /// supported options.
 /// The API is fluent, and the options are sorted in alphabetical order below.
 /// The options can be exposed to the LLVM command line by registering them
-/// with `MlirOptMainConfig::registerCLOptions(DialectRegistry &);` and creating
-/// a config using `auto config = MlirOptMainConfig::createFromCLOptions();`.
+/// with `MlirOptMainConfig::registerCLOptions();` and creating a config using
+/// `auto config = MlirOptMainConfig::createFromCLOptions();`.
 class MlirOptMainConfig {
 public:
   /// Register the options as global LLVM command line options.
-  static void registerCLOptions(DialectRegistry &dialectRegistry);
+  static void registerCLOptions();
 
   /// Create a new config with the default set from the CL options.
   static MlirOptMainConfig createFromCLOptions();
index 053769f..4377ebe 100644 (file)
@@ -14,7 +14,6 @@ add_mlir_library(MLIRIR
   BuiltinTypeInterfaces.cpp
   Diagnostics.cpp
   Dialect.cpp
-  DialectPlugin.cpp
   DialectResourceBlobManager.cpp
   Dominance.cpp
   ExtensibleDialect.cpp
diff --git a/mlir/lib/IR/DialectPlugin.cpp b/mlir/lib/IR/DialectPlugin.cpp
deleted file mode 100644 (file)
index 64b2874..0000000
+++ /dev/null
@@ -1,53 +0,0 @@
-//===- lib/IR/DialectPlugin.cpp - Load Dialect Plugins --------------------===//
-//
-// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
-
-#include "mlir/IR/DialectPlugin.h"
-#include "llvm/Support/raw_ostream.h"
-
-#include <cstdint>
-
-using namespace mlir;
-
-llvm::Expected<DialectPlugin> DialectPlugin::load(const std::string &filename) {
-  std::string error;
-  auto library =
-      llvm::sys::DynamicLibrary::getPermanentLibrary(filename.c_str(), &error);
-  if (!library.isValid())
-    return llvm::make_error<llvm::StringError>(
-        Twine("Could not load library '") + filename + "': " + error,
-        llvm::inconvertibleErrorCode());
-
-  DialectPlugin plugin{filename, library};
-
-  // mlirGetDialectPluginInfo should be resolved to the definition from the
-  // plugin we are currently loading.
-  intptr_t getDetailsFn =
-      (intptr_t)library.getAddressOfSymbol("mlirGetDialectPluginInfo");
-
-  if (!getDetailsFn)
-    return llvm::make_error<llvm::StringError>(
-        Twine("Plugin entry point not found in '") + filename,
-        llvm::inconvertibleErrorCode());
-
-  plugin.info =
-      reinterpret_cast<decltype(mlirGetDialectPluginInfo) *>(getDetailsFn)();
-
-  if (plugin.info.apiVersion != MLIR_PLUGIN_API_VERSION)
-    return llvm::make_error<llvm::StringError>(
-        Twine("Wrong API version on plugin '") + filename + "'. Got version " +
-            Twine(plugin.info.apiVersion) + ", supported version is " +
-            Twine(MLIR_PLUGIN_API_VERSION) + ".",
-        llvm::inconvertibleErrorCode());
-
-  if (!plugin.info.registerDialectRegistryCallbacks)
-    return llvm::make_error<llvm::StringError>(
-        Twine("Empty entry callback in plugin '") + filename + "'.'",
-        llvm::inconvertibleErrorCode());
-
-  return plugin;
-}
index 82a69ed..5ca9b41 100644 (file)
@@ -3,7 +3,6 @@ add_mlir_library(MLIRPass
   Pass.cpp
   PassCrashRecovery.cpp
   PassManagerOptions.cpp
-  PassPlugin.cpp
   PassRegistry.cpp
   PassStatistics.cpp
   PassTiming.cpp
diff --git a/mlir/lib/Pass/PassPlugin.cpp b/mlir/lib/Pass/PassPlugin.cpp
deleted file mode 100644 (file)
index fc39ce5..0000000
+++ /dev/null
@@ -1,53 +0,0 @@
-//===- lib/Passes/PassPlugin.cpp - Load Plugins for PR Passes ---===//
-//
-// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
-
-#include "mlir/Pass/PassPlugin.h"
-#include "llvm/Support/raw_ostream.h"
-
-#include <cstdint>
-
-using namespace mlir;
-
-llvm::Expected<PassPlugin> PassPlugin::load(const std::string &filename) {
-  std::string Error;
-  auto library =
-      llvm::sys::DynamicLibrary::getPermanentLibrary(filename.c_str(), &Error);
-  if (!library.isValid())
-    return llvm::make_error<llvm::StringError>(
-        Twine("Could not load library '") + filename + "': " + Error,
-        llvm::inconvertibleErrorCode());
-
-  PassPlugin plugin{filename, library};
-
-  // mlirGetPassPluginInfo should be resolved to the definition from the plugin
-  // we are currently loading.
-  intptr_t getDetailsFn =
-      (intptr_t)library.getAddressOfSymbol("mlirGetPassPluginInfo");
-
-  if (!getDetailsFn)
-    return llvm::make_error<llvm::StringError>(
-        Twine("Plugin entry point not found in '") + filename,
-        llvm::inconvertibleErrorCode());
-
-  plugin.info =
-      reinterpret_cast<decltype(mlirGetPassPluginInfo) *>(getDetailsFn)();
-
-  if (plugin.info.apiVersion != MLIR_PLUGIN_API_VERSION)
-    return llvm::make_error<llvm::StringError>(
-        Twine("Wrong API version on plugin '") + filename + "'. Got version " +
-            Twine(plugin.info.apiVersion) + ", supported version is " +
-            Twine(MLIR_PLUGIN_API_VERSION) + ".",
-        llvm::inconvertibleErrorCode());
-
-  if (!plugin.info.registerPassRegistryCallbacks)
-    return llvm::make_error<llvm::StringError>(
-        Twine("Empty entry callback in plugin '") + filename + "'.'",
-        llvm::inconvertibleErrorCode());
-
-  return plugin;
-}
index ecbf4aa..dfc5117 100644 (file)
 #include "mlir/IR/BuiltinOps.h"
 #include "mlir/IR/Diagnostics.h"
 #include "mlir/IR/Dialect.h"
-#include "mlir/IR/DialectPlugin.h"
 #include "mlir/IR/Location.h"
 #include "mlir/IR/MLIRContext.h"
 #include "mlir/Parser/Parser.h"
 #include "mlir/Pass/Pass.h"
 #include "mlir/Pass/PassManager.h"
-#include "mlir/Pass/PassPlugin.h"
 #include "mlir/Support/FileUtilities.h"
 #include "mlir/Support/Timing.h"
 #include "mlir/Support/ToolUtilities.h"
@@ -103,41 +101,15 @@ struct MlirOptMainConfigCLOptions : public MlirOptMainConfig {
         cl::desc("Run the verifier after each transformation pass"),
         cl::location(verifyPassesFlag), cl::init(true));
 
-    static cl::list<std::string> passPlugins(
-        "load-pass-plugin", cl::desc("Load passes from plugin library"));
-    /// Set the callback to load a pass plugin.
-    passPlugins.setCallback([&](const std::string &pluginPath) {
-      auto plugin = PassPlugin::load(pluginPath);
-      if (!plugin) {
-        errs() << "Failed to load passes from '" << pluginPath
-               << "'. Request ignored.\n";
-        return;
-      }
-      plugin.get().registerPassRegistryCallbacks();
-    });
-
-    static cl::list<std::string> dialectPlugins(
-        "load-dialect-plugin", cl::desc("Load dialects from plugin library"));
-    this->dialectPlugins = std::addressof(dialectPlugins);
-
     static PassPipelineCLParser passPipeline("", "Compiler passes to run", "p");
     setPassPipelineParser(passPipeline);
   }
-
-  /// Set the callback to load a dialect plugin.
-  void setDialectPluginsCallback(DialectRegistry &registry);
-
-  /// Pointer to static dialectPlugins variable in constructor, needed by
-  /// setDialectPluginsCallback(DialectRegistry&).
-  cl::list<std::string> *dialectPlugins = nullptr;
 };
 } // namespace
 
 ManagedStatic<MlirOptMainConfigCLOptions> clOptionsConfig;
 
-void MlirOptMainConfig::registerCLOptions(DialectRegistry &registry) {
-  clOptionsConfig->setDialectPluginsCallback(registry);
-}
+void MlirOptMainConfig::registerCLOptions() { *clOptionsConfig; }
 
 MlirOptMainConfig MlirOptMainConfig::createFromCLOptions() {
   return *clOptionsConfig;
@@ -162,19 +134,6 @@ MlirOptMainConfig &MlirOptMainConfig::setPassPipelineParser(
   return *this;
 }
 
-void MlirOptMainConfigCLOptions::setDialectPluginsCallback(
-    DialectRegistry &registry) {
-  dialectPlugins->setCallback([&](const std::string &pluginPath) {
-    auto plugin = DialectPlugin::load(pluginPath);
-    if (!plugin) {
-      errs() << "Failed to load dialect plugin from '" << pluginPath
-             << "'. Request ignored.\n";
-      return;
-    };
-    plugin.get().registerDialectRegistryCallbacks(registry);
-  });
-}
-
 /// Set the ExecutionContext on the context and handle the observers.
 class InstallDebugHandler {
 public:
@@ -406,7 +365,7 @@ LogicalResult mlir::MlirOptMain(int argc, char **argv, llvm::StringRef toolName,
   InitLLVM y(argc, argv);
 
   // Register any command line options.
-  MlirOptMainConfig::registerCLOptions(registry);
+  MlirOptMainConfig::registerCLOptions();
   registerAsmPrinterCLOptions();
   registerMLIRContextCLOptions();
   registerPassManagerCLOptions();
index 2d021d5..c430569 100644 (file)
@@ -79,10 +79,8 @@ add_mlir_tool(mlir-opt
 
   DEPENDS
   ${LIBS}
-  SUPPORT_PLUGINS
   )
 target_link_libraries(mlir-opt PRIVATE ${LIBS})
 llvm_update_compile_flags(mlir-opt)
 
 mlir_check_all_link_libraries(mlir-opt)
-export_executable_symbols_for_plugins(mlir-opt)