masahi opened a new pull request, #13605:
URL: https://github.com/apache/tvm/pull/13605

   https://github.com/apache/tvm/pull/13560 is supposed to change only the 
order in which buffers are visited, but the created `buffer_alloc_recorder_` 
misses buffers associated with `AllocateConst` nodes. These buffers appear only 
in `read` or `match_buffer` regions, so they are not caught by existing 
visitors for `BufferLoadNode` and `BufferStoreNode`.
   
   Since constant buffers are not correctly handled by 
`PlanAndUpdateBufferAllocationLocation` now, tuning on Hexagon is currently 
broken with this error. This PR fixes this.
   
   ```
   E             File 
"/home/fparizi/src/tvm_src/src/tir/transforms/plan_update_buffer_allocation_location.cc",
 line 163
   E           TVMError: 
   E           ---------------------------------------------------------------
   E           An error occurred during the execution of TVM.
   E           For more information, please see: 
https://tvm.apache.org/docs/errors.html
   E           ---------------------------------------------------------------
   E             Check failed: (buffer_data_to_buffer_.count(source_var)) is 
false:
   ```


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