zhiics commented on issue #5410: URL: https://github.com/apache/incubator-tvm/pull/5410#issuecomment-618079321
@mbrookhart @comaniac The problem is not just about current pass that will be invoked, i.e. `AnnotateTarget`. We will update the module in the pass manager for function passes here: https://github.com/apache/incubator-tvm/blob/f5c9bc93883d426284057657463942dfdfef2fd8/src/relay/ir/transform.cc#L140-L142 This will indirectly call InferType here: https://github.com/apache/incubator-tvm/blob/f5c9bc93883d426284057657463942dfdfef2fd8/src/ir/module.cc#L225 Therefore, I think we may need to change `InferType` to non-recursive version first. Otherwise, many passes will still fail (indirectly/magically). ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
