anijain2305 commented on a change in pull request #5153: Adding support for QNN subtract op URL: https://github.com/apache/incubator-tvm/pull/5153#discussion_r398786009
########## 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: I think DataType::Int(32) is more or as informative as fullPrecisionInt32. ---------------------------------------------------------------- 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
