[GitHub] [incubator-tvm-vta] remotego commented on pull request #8: [Hardware][Xilinx] explicitly specify acc dep distance to avoid hidden pitfall

remotego commented on pull request #8: URL: https://github.com/apache/incubator-tvm-vta/pull/8#issuecomment-621601302 > Perhaps one way to look at this is to start with DISTANCE=3 by default. And if the II>1 for the GEMM, we issue a warning telling the user to increase the distance to 4.

[GitHub] [incubator-tvm-vta] remotego commented on pull request #8: [Hardware][Xilinx] explicitly specify acc dep distance to avoid hidden pitfall

remotego commented on pull request #8: URL: https://github.com/apache/incubator-tvm-vta/pull/8#issuecomment-621593259 > Thanks @zhanghaohit, for catching this error. I agree with the fact that the dependence should not be hardcoded. However in order to not to add too many parameters in