vinx13 commented on pull request #42: URL: https://github.com/apache/tvm-rfcs/pull/42#issuecomment-955093077
I agree with @spectrometerHBH that changing the IR design may introduce problems. To reuse the code but BufferLoad/Store, we can introduce auxiliary functions or construct auxiliary data structure on the fly. The `BufferPointer` now is `PrimExpr`, we must be careful it doesn't appear in other places. We intentionally keep things restricted so that buffer do not alias and not addressable(e.g. no concrete ptr). During rewriting, we don't deal with pointers explicitly, I'd prefer keeping the concept of pointer abstract, to prevent potential issue of aliasing. -- 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]
