[Mlir-commits] [mlir] [mlir] Clone attrs of unregistered dialect ops (PR #151847)

[Matthias Springer]

================
@@ -734,6 +734,12 @@ Operation *Operation::clone(IRMapping &mapper, CloneOptions options) {
   // Create the new operation.
   auto *newOp = create(getLoc(), getName(), getResultTypes(), operands, attrs,
                        getPropertiesStorage(), successors, getNumRegions());
+
+  // The builder overwrites the opaque properties with the attributes; hence,
+  // copy them after building the operation.
+  if (!isRegistered())
----------------
matthias-springer wrote:

What about registered ops? How come this is only needed for unregistered ops?


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