anijain2305 commented on pull request #6430: URL: https://github.com/apache/incubator-tvm/pull/6430#issuecomment-690600943
Yes, that makes sense. Thanks for explaining! Yes, if we want to go through something like `TempOpAttr`, we need that functionality in C++. Is your current PR implementation easier or more scalable then? I think in your current PR, you can define the TVMPackedFunc counterpart in the `PreProcessModule` C++ itself? ---------------------------------------------------------------- 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: us...@infra.apache.org
