tqchen commented on PR #15983: URL: https://github.com/apache/tvm/pull/15983#issuecomment-1799372140
I understand the error scenario you described. The main issue here is that IntImm and FloatImm are compile time construct that are not defined as partof the runtime (because they are coupled as Expr). As a result, the implementation won't allow us to support the case where only tvm_runtime is available, which is a common case to support. A better approach would be to enable boxed `BoxInt` and `Float` that are decoupled from IntImm and FloatImm, and migrate the existing usage of IntImm to those runtime boxed cases. Given tuple of two prim values are less commonly used and we only recently supported the prim values. I think it is better for us to not support the case for now, so we can wait for longer term solution that separates out dedicated runtime boxed values, in which case automatic conversion would makes sense and there won't be limitations that it won't work in a runtime onlys etting. -- 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]
