[Mlir-commits] [mlir] [mlir][bytecode] Implements back deployment capability for MLIR dialects (PR #70724)

[Jeff Niu]

================
@@ -400,6 +400,10 @@ class DialectBytecodeWriter {
 
   /// Return the bytecode version being emitted for.
   virtual int64_t getBytecodeVersion() const = 0;
+
+  /// Retrieve the dialect version by name if available.
+  virtual FailureOr<const DialectVersion *>
+  getDialectVersion(StringRef dialectName) const = 0;
----------------
Mogball wrote:

suggestion: can you also add versions of this API that accept a `Dialect *`, or one like `getDialectVersion<MyDialect>()`? MLIR APIs typical expose both: strings for flexibility, and a template version for more idiomatic code.

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