Skip to content

Conversation

@WillFroom
Copy link
Contributor

Fix after #140963

@llvmbot llvmbot added mlir:core MLIR Core Infrastructure mlir labels May 29, 2025
@llvmbot
Copy link
Member

llvmbot commented May 29, 2025

@llvm/pr-subscribers-mlir

@llvm/pr-subscribers-mlir-core

Author: Will Froom (WillFroom)

Changes

Fix after #140963


Full diff: https://github.com/llvm/llvm-project/pull/141915.diff

1 Files Affected:

  • (modified) mlir/include/mlir/IR/DialectRegistry.h (+2)
diff --git a/mlir/include/mlir/IR/DialectRegistry.h b/mlir/include/mlir/IR/DialectRegistry.h
index 7bcf1eda7c636..b7d3e5d67e6d7 100644
--- a/mlir/include/mlir/IR/DialectRegistry.h
+++ b/mlir/include/mlir/IR/DialectRegistry.h
@@ -145,6 +145,8 @@ class DialectRegistry {
   explicit DialectRegistry();
   DialectRegistry(const DialectRegistry &) = delete;
   DialectRegistry &operator=(const DialectRegistry &other) = delete;
+  DialectRegistry(DialectRegistry &&) = default;
+  DialectRegistry &operator=(DialectRegistry &&other) = default;
 
   template <typename ConcreteDialect>
   void insert() {

@WillFroom WillFroom merged commit ebe25d8 into llvm:main May 29, 2025
11 of 14 checks passed
svkeerthy pushed a commit that referenced this pull request May 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mlir:core MLIR Core Infrastructure mlir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants