[mlir] Update comment about propertiesAttr (NFC)

[Mogball]

Stacked PRs:

• ->[mlir] Update comment about propertiesAttr (NFC) #89631

---

[mlir] Update comment about propertiesAttr (NFC)

The comment is misleading because propertiesAttr is not actually
ignored when the operation isn't unregistered.

[llvmbot]

@llvm/pr-subscribers-mlir

Author: Jeff Niu (Mogball)

Changes

Stacked PRs:

• ->#89631

---

[mlir] Update comment about propertiesAttr (NFC)

The comment is misleading because propertiesAttr is not actually
ignored when the operation isn't unregistered.

---

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

1 Files Affected:

• (modified) mlir/include/mlir/IR/OperationSupport.h (+6-3)

diff --git a/mlir/include/mlir/IR/OperationSupport.h b/mlir/include/mlir/IR/OperationSupport.h
index 2c1c490aac49b8..2c6e8253b4327a 100644
--- a/mlir/include/mlir/IR/OperationSupport.h
+++ b/mlir/include/mlir/IR/OperationSupport.h
@@ -960,9 +960,12 @@ struct OperationState {
   /// Regions that the op will hold.
   SmallVector<std::unique_ptr<Region>, 1> regions;
 
-  // If we're creating an unregistered operation, this Attribute is used to
-  // build the properties. Otherwise it is ignored. For registered operations
-  // see the `getOrAddProperties` method.
+  // This Attribute is used to opaquely construct the properties of the
+  // operation. If we're creating an unregistered operation, the Attribute is
+  // used as-is as the Properties storage of the operation. Otherwise, the
+  // operation properties are constructed opaquely using its
+  // `setPropertiesFromAttr` hook. Note that `getOrAddProperties` is the
+  // preferred method to construct properties from C++.
   Attribute propertiesAttr;
 
 private:

[llvmbot]

@llvm/pr-subscribers-mlir-core

Author: Jeff Niu (Mogball)

Changes

Stacked PRs:

• ->#89631

---

[mlir] Update comment about propertiesAttr (NFC)

The comment is misleading because propertiesAttr is not actually
ignored when the operation isn't unregistered.

---

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

1 Files Affected:

• (modified) mlir/include/mlir/IR/OperationSupport.h (+6-3)

diff --git a/mlir/include/mlir/IR/OperationSupport.h b/mlir/include/mlir/IR/OperationSupport.h
index 2c1c490aac49b8..2c6e8253b4327a 100644
--- a/mlir/include/mlir/IR/OperationSupport.h
+++ b/mlir/include/mlir/IR/OperationSupport.h
@@ -960,9 +960,12 @@ struct OperationState {
   /// Regions that the op will hold.
   SmallVector<std::unique_ptr<Region>, 1> regions;
 
-  // If we're creating an unregistered operation, this Attribute is used to
-  // build the properties. Otherwise it is ignored. For registered operations
-  // see the `getOrAddProperties` method.
+  // This Attribute is used to opaquely construct the properties of the
+  // operation. If we're creating an unregistered operation, the Attribute is
+  // used as-is as the Properties storage of the operation. Otherwise, the
+  // operation properties are constructed opaquely using its
+  // `setPropertiesFromAttr` hook. Note that `getOrAddProperties` is the
+  // preferred method to construct properties from C++.
   Attribute propertiesAttr;
 
 private: