shoubhik commented on a change in pull request #5153: Adding support for QNN subtract op URL: https://github.com/apache/incubator-tvm/pull/5153#discussion_r398850100
########## File path: src/relay/qnn/op/mul.cc ########## @@ -71,31 +59,35 @@ Expr QnnMulCanonicalize(const Attrs& attrs, const Array<Expr>& new_args, which is essentially a requantization of tensor Q' into tensor Q_c. */ - auto lhs_shifted = Cast(lhs, DataType::Int(32)); - auto rhs_shifted = Cast(rhs, DataType::Int(32)); + auto lhs_shifted = Cast(args.lhs, fullPrecisionInt32); Review comment: One compromise could be just to name them `Int32` or `Float32` which carries the necessary information and also has the safety associated. For now I have made the change have added the names Int32 and Float32 so it both intuitive and easy to change. ---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
