tqchen opened a new pull request, #19662: URL: https://github.com/apache/tvm/pull/19662
TVM can rely on tvm-ffi's JSON graph serialization helpers directly instead of routing through TVM-side `node.SaveJSON`/`node.LoadJSON` registry entries. This changes `tvm.ir` save/load to call `tvm_ffi.serialization` with `tvm_version` metadata, removes the C++ registry wrapper, and moves the disco debug object path to `ffi::ToJSONGraph`/`ffi::FromJSONGraph` plus JSON parse/stringify. The disco Python wrappers now declare Python attribute storage explicitly for `DRef` and `Session` so `DPackedFunc`/`DModule` and method caches continue to work with the current tvm-ffi object model. The socket address helper also normalizes `localhost` consistently across constructors so the disco socket debug round-trip can bind an IPv4 socket when `localhost` resolves to IPv6 first. Validated locally in an isolated worktree build with `ninja -C build tvm_compiler tvm_runtime_extra`, targeted IR/target tests, `tests/python/disco/test_session.py::test_string_obj`, import smoke, and touched-file pre-commit. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
