adstraw commented on PR #13719: URL: https://github.com/apache/tvm/pull/13719#issuecomment-1382314839
Nice work. Just wanted to note that we will need to (and have plans to) put our heads together on a unified approach to DMA lowering. Right now, the async DMA flow stems from TIR annotations added by the InjectSoftwarePipe pass. This PR suggests a tensorization approach for synchronous DMA. I feel we could benefit from a unified approach. I recently deleted an old (TE based) tensorization approach for sync DMA scheduling. Glad to have this new TIR based version and it may in fact be the path forward for a unified approach to DMA lowering. Let's put our heads together and get the right design. -- 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]
