Lunderberg commented on PR #17108: URL: https://github.com/apache/tvm/pull/17108#issuecomment-2176736539
I don't remember that discussion, but I'd be interested in seeing the minutes from it. The main roadblocks to it are ones that I've resolved in previous PRs: * Accidentally scheduling a host function, fixed in https://github.com/apache/tvm/pull/16958 * Accidentally scheduling the output of `ComputePrimValue`, fixed in https://github.com/apache/tvm/pull/17032. * Accidentally de-duplicating functions that differ by vdevice, only one of which should be scheduled for GPU, fixed in https://github.com/apache/tvm/pull/16959. With those fixes applied, I think it's now a pretty big usability benefit to include it in the lowering pipeline. -- 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]
