[Mlir-commits] [mlir] [MLIR] Add inherent attribute visitor (PR #217881)

[Mehdi Amini]

================
@@ -107,6 +107,7 @@ class OperationName {
   // class is defined below.
   using PopulateDefaultAttrsFn =
       llvm::unique_function<void(const OperationName &, NamedAttrList &) const>;
+  using InherentAttrVisitor = llvm::function_ref<void(StringAttr, Attribute &)>;
----------------
joker-eph wrote:

Indeed! I had DictionaryAttr in mind and missed that this would add an unnecessary conversion...

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