tvm-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [incubator-tvm-vta] tmoreau89 commented on pull request #8: [Hardware][Xilinx] explicitly specify acc dep distance to avoid hidden pitfall
Date Thu, 11 Jun 2020 03:42:08 GMT

tmoreau89 commented on pull request #8:
URL: https://github.com/apache/incubator-tvm-vta/pull/8#issuecomment-642387343


   @zhanghaohit I think that the warning approach is a reasonable compromise between user-defined
flexibility and making sure that they set the value correctly.
   
   To echo @liangfu , it would be ideal to derive the value in pkg_config based on simple
rules of FPGA target, and matrix multiplication size. We can tweak those rules if say we discover
that the dependence distance needs to be increased.
   
   If you agree I suggest we go ahead with these changes.


----------------------------------------------------------------
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:
users@infra.apache.org



Mime
View raw message