youxiudeshouyeren opened a new issue, #16490: URL: https://github.com/apache/tvm/issues/16490
In the debug_executor's `debug_get_output` function, the out parameter can be set to None, but in the C++ implementation of the function, there is no check to see if the out parameter is None, which can result in the program reporting an error without a prompt. The details are as follows: 1. `debug_get_output` function in tvm/python/tvm/contrib/debugger/debug_executor.py  2. C++ implementation in tvm/src/runtime/graph_executor/debug/graph_executor_debug.cc. This function has no return value and copies Ndarray directly to the out parameter in python.  3. When out is None, this will raise a direct error: segment fault  I would like to raise a pr to address this issue. Now there are two choices: 1. check at python level that this out parameter cannot be None. 2. check the number of arguments in C++ and return Ndarray directly when out is None. This operation has the same behavior as get_output function in executor. I prefer the second choice. Thanks for reading and what do you suggest？ @Hzfengsy -- 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]
