slyubomirsky commented on code in PR #14282:
URL: https://github.com/apache/tvm/pull/14282#discussion_r1134719166
##########
src/runtime/relax_vm/builtin.cc:
##########
@@ -380,6 +380,40 @@
TVM_REGISTER_GLOBAL("vm.builtin.make_tuple").set_body([](TVMArgs args, TVMRetVal
*rv = arr;
});
+TVM_REGISTER_GLOBAL("vm.builtin.tensor_to_shape").set_body_typed([](NDArray
data) {
+ NDArray arr = data;
+ if (data->device.device_type != kDLCPU) {
+ arr = data.CopyTo(DLDevice{kDLCPU, 0});
Review Comment:
Not sure where this might come up. I wouldn't be surprised if, say, we
import an ONNX model (this is where this use-case first came up) that we mean
to run on GPU and every tensor (including those that stand for shapes) is
stored on GPU.
--
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]