apeskov commented on PR #12323: URL: https://github.com/apache/tvm/pull/12323#issuecomment-1223805016
The word "Global" means that it's a global singleton. Changing semantic to thread local will confuse developers. Moreover, this patch works only because you initialise VirtualMachine modules form separate threads. That is not mandatory behaviour. Customer may create a set of VirtualMachine in main thread and after that assign them to worker sub threads. This change is good enough to demonstrate possibility of advance memory management in case of multi instance execution. But it cannot be merged as is. -- 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]
