masahi commented on pull request #5689: URL: https://github.com/apache/incubator-tvm/pull/5689#issuecomment-635658618
> @masahi I just checked with this PR. The unit test hits the line you pointed out twice. hmm something seems off to me. On the dnnl fused mobilenet tests, I think all params in the network should go through that visitor. In https://github.com/apache/incubator-tvm/pull/5310, @zhiics improved how to handle big constants, which made running fused mobilenet possible. If the visitor is called only twice, we shouldn't have had such issues at all. ---------------------------------------------------------------- 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]
