AndrewZhaoLuo commented on pull request #8363: URL: https://github.com/apache/tvm/pull/8363#issuecomment-870916059
Ok folks ready for review. This works with llvm and metal on m1 mac with "spawn" multiprocessing enabled. I also added a test to make sure other multiprocessing tests work. The key assumption is when invoking the `Target::Target(Target target, Target host)` constructor it's ok if `target->host` and `host` refer to the same underlying target. This appears to be the intention with the line `n->host.same_as(host)`. However `spawn()` breaks any pointer equality so we need to have some sort of deep equals. This does not exist so for now I'm just comparing the human readable strings instead. -- 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]
