tqchen commented on PR #228: URL: https://github.com/apache/tvm-ffi/pull/228#issuecomment-3490897252
please split the core changes `function_details` into a separate PR. likely stl should go into extra. The main reason is that while stl can be helpful sometimes, the tradeoff is when present in cases like object, we can no longer to near zero cost access via directly ptr and access, so we should always encourage ffi types when possible -- 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]
