tmoreau89 commented on a change in pull request #9:
URL: https://github.com/apache/incubator-tvm-vta/pull/9#discussion_r443089629



##########
File path: include/vta/hw_spec.h
##########
@@ -404,7 +259,7 @@ typedef struct {
   /*! \brief Destination index (indexes accum buffer) */
   uint32_t dst_idx    : VTA_LOG_ACC_BUFF_DEPTH;
   /*! \brief Source index (indexes input buffer for GEMM or accum buffer for 
ALU) */
-  uint32_t src_idx    : VTA_LOG_INP_BUFF_DEPTH;
+  uint32_t src_idx    : VTA_LOG_ACC_BUFF_DEPTH;

Review comment:
       We may want to set it to max(VTA_LOG_ACC_BUFF_DEPTH, 
VTA_LOG_INP_BUFF_DEPTH)




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


Reply via email to