mbaret commented on pull request #8806: URL: https://github.com/apache/tvm/pull/8806#issuecomment-902997745
I appreciate this doesn't exactly trivialize the patch, but it's actually 'only' 4k lines because this is on top of https://github.com/apache/tvm/pull/8795. I've got this as draft until that PR gets merged to give visibility, but I'm not expecting this to necessarily get reviewed until the dependent PR gets merged. The particular problem we have splitting this PR up is that we need to introduce an operator alongside some of the compiler support so that we can test it. That's not to say it's impossible though, and if you still think 4k is too much I'm happy to explore some options to reduce it further. -- 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]
