You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2022/05/05 03:16:58 UTC

[GitHub] [tvm] cee1 commented on pull request #11155: fix accuracy issue of misalignment between arm's q_multiply_shift and x86's

cee1 commented on PR #11155:
URL: https://github.com/apache/tvm/pull/11155#issuecomment-1118125802

   > Why do we think the DEFAULT path is "accurate" here ?
   
   DEFAULT path is (somewhat) consistent with QAT, specifically, the rounding behavior within fake quantization.
    
   For DEFAULT path, only one right shift is required, and the (only one) rounding happens there.
   
   For NEON path, the (possible) extra rounding behavior is introduced by NEON instruction `sqrdmulh`, which has some side-effect (see https://discuss.tvm.apache.org/t/quantization-aligning-result-of-tvm-to-torchs/12225/2?u=cee1 )


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org