csullivan commented on a change in pull request #7488:
URL: https://github.com/apache/tvm/pull/7488#discussion_r583808824
##########
File path: src/runtime/rpc/rpc_endpoint.h
##########
@@ -147,8 +146,7 @@ class RPCEndpoint {
* \param ctx_from The source context.
* \param type_hint Hint of content data type.
*/
- void CopyFromRemote(void* from, size_t from_offset, void* to, size_t
to_offset, size_t nbytes,
- TVMContext ctx_from, DLDataType type_hint);
+ void CopyFromRemote(DLTensor* from, void* to_bytes, uint64_t nbytes);
Review comment:
Agreed on const void* for data field protection. Prefer to provide as
much direction as possible on ownership, but there's only so much one can
indicate with c pointers.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
