xqdan commented on pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#issuecomment-626509149
> > Binds in some pass function is not clean for round trip dump ir, how do we deal with it? > > Do you mean that `buffer_bind` is not necessary to print after the pass `storage_flatten`? > Yes, the buffer no longer exists after the flatten pass. All we have is that Var(buffer->data) in Load and Store. But we still need a place to define those vars, and that's why we still have buffer_bind, where we define buffer as well as buffer->data, in low-level functions > > Furthermore, maybe one day we can use BufferLoad/ BufferStore after the flatten and use buffer from the beginning to the end. https://github.com/apache/incubator-tvm/blob/master/src/driver/driver_api.cc#L139 that's i'm asking, seems binds is embedding into ir at the beginning, looks good ---------------------------------------------------------------- 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
