[Mlir-commits] [mlir] [MLIR] Add C-API for parsing bytecode (PR #83825)

[llvmlistbot at llvm.org]

================
@@ -43,7 +43,7 @@ class AttrTypeBytecodeReader {
                     CallableT, std::function<LogicalResult(
                                    DialectBytecodeReader &, StringRef, T &)>>,
                 bool> = true>
-  static std::unique_ptr<AttrTypeBytecodeReader<T>>
+  static std::shared_ptr<AttrTypeBytecodeReader<T>>
----------------
SpriteOvO wrote:

Makes sense. In the latest commit, I made the read config optional in `ParserConfig` so that we could avoid using `shared_ptr` here.

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