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

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

mwillsey edited a comment 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 DLContext. 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