Thanks @jackwish 
This is a very good analysis. Everything makes sense. I upvote for restricting 
to `(u)int8` for now for `Quantize` and `Dequantize`.

If in future, we see `(u)int16`, we can tackle then. `int32` is highly unlikely 
(why not just go to `FP32` as you say).

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/dmlc/tvm/issues/3591#issuecomment-515112105

Reply via email to