[Mlir-commits] [mlir] [mlir] Deprecate NamedAttrList(std::nullopt_t) (NFC) (PR #149544)

[Kazu Hirata]

================
@@ -802,6 +802,7 @@ class NamedAttrList {
   using size_type = size_t;
 
   NamedAttrList() : dictionarySorted({}, true) {}
+  LLVM_DEPRECATED("Use NamedAttrList() instead", "NamedAttrList()")
   NamedAttrList(std::nullopt_t none) : NamedAttrList() {}
----------------
kazutakahirata wrote:

Thanks for the review!  Hmm.  `{}` is short.  `OpPrintingFlags()` is descriptive. `nullptr_t` is neither short nor descriptive.  `nullptr_t` would work well in the perfect-forwarding context, however.

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