Thank you @manupa-arm for the proposal. 

What you described is closest to the "tag" field of the storage scope, which we 
put as part of the pointer and associate with most of the storage. I think it 
would be great to think in terms of this direction.

Specifically, when we have a clear distinction of memory types, "sram", 
"shared_memory", "texture". We will use the special memory tag to indicate the 
type of such kind of memory.

The proposed pinned memory field, however, have multiple of such candidates. In 
my understanding tries to serve as an additional hints to follow up passes. If 
the intended use is multiple candidate hints, then the tag is perhaps no longer 
approperiate because it is used to indicate one kind of special memory that we 
place the data to.

In that case, I think it could be closer to an additional annotation field 
https://github.com/apache/tvm/blob/main/include/tvm/tir/stmt.h#L817 that stores 
additional attribute hints about the particular node. This would also help 
reduce the possible indirection introduced by AttrStmt.







-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/apache/tvm-rfcs/pull/23#issuecomment-903112540

Reply via email to