tqchen opened a new issue #10505:
URL: https://github.com/apache/tvm/issues/10505


   This is a tracking issue created for followup tracking items in 
https://github.com/apache/tvm/pull/9727.
   
   #9727 contains a major refactor that migrates the low level Load/Store 
access to BufferLoad/Store. The particular PR takes many rounds of reviews and 
still have quite a few points that would benefit from systematic clarification, 
cleanup and further polish.
   
   Given that there is an important followup(of layout transform) that depend 
on the change, and the PR is already at a state where incremental changes can 
further block the progress as other concurrent changes are made.  I took the 
deliberation to merge PR, then continue to do post-hoc shepherding. This 
decision is supposed to happen in rare circumstance with clear followup actions 
to backup the decisions. 
   
   This is the tracking issue for followup shepherd actions
   
   - [ ] Clarify the buffer access rule, aliasing and allocation rules
   - [ ] Find alternative to `preflatten_buffer_map` (possible via buffer 
redeclaration)
   - [ ] Discuss and narrow down the scope of `address_of`, ideally to 1D acess 
   - [ ] Discuss and clarify the scope of vector buffer indexing rule
   - [ ] Post-hoc Revisit key passes involved
       - [ ] CodegenLLVM
       - [ ] StorageRewrite
       - [ ] Vectorize
       - [ ] VectorPointerTypeRewrite
       - [ ] Other passes that are involved in the refactoring
   
   
   


-- 
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]


Reply via email to