tqchen opened a new pull request, #700: URL: https://github.com/apache/tvm-ffi/pull/700
## Rationale The TVMFFIAny-backed Optional layout is the intended forward ABI, but Optional<Function> grew from one pointer to one TVMFFIAny cell. ModuleObj function lookup is a C++ ABI boundary whose symbols do not encode return types, so retaining Optional<Function> makes old and new callers disagree about return storage. Use nullable Function for both ModuleObj lookup overloads. Function remains pointer-sized and preserves the historical null representation, while Optional itself and the String metadata APIs retain their current ABI. ## Change - Return nullable Function from direct and import-aware module lookup. - Update library and ORCJIT module implementations plus C++ examples. -- 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]
