You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2020/08/04 21:25:00 UTC

[GitHub] [incubator-tvm] jroesch commented on issue #6197: [Rust] TVMError: Check failed: type_code_ == kDLInt (1 vs. 0) : expected int but get uint

jroesch commented on issue #6197:
URL: https://github.com/apache/incubator-tvm/issues/6197#issuecomment-668833017


   @12101111 agreed I think this is another leftover from the previous version which I've been working on cleaning up and refactoring to bring up to date with modern TVM. My feeling is the graph runtime should use `tvm_sys::Context` as its public interface and only convert to the low level `tvm_sys::ffi` structs in a single location where we can be sure to correctly match the types as its very error prone and easy to pick the wrong integer types as they will work "most of the time". 


----------------------------------------------------------------
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