[GitHub] [incubator-tvm] mwillsey commented on pull request #6545: [Rust] change Context.device_id from usize to c_int (#6528)

[GitBox <gi...@apache.org>]

mwillsey commented on pull request #6545:
URL: https://github.com/apache/incubator-tvm/pull/6545#issuecomment-698480657


   @intheworld could you say a little more about why this change is needed? I think the point of Context is that it's a "safer" Rust version of TVMContext. As long as you check that `device_id` is positive, the conversion should be fine (typed conversion, not just casting). I'd rather have usize (or u32) on the Rust side, since you don't have to worry about negative device ids


----------------------------------------------------------------
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:
users@infra.apache.org