junrushao1994 commented on a change in pull request #6103:
URL: https://github.com/apache/incubator-tvm/pull/6103#discussion_r459183926
##########
File path: src/auto_scheduler/compute_dag.cc
##########
@@ -126,6 +555,7 @@ class FlopEstimator : public ExprFunctor<double(const
PrimExpr& n)> {
fail_ = true;
break;
}
+ cur_type_code_ = pop->output_dtype(0).code();
Review comment:
@merrymercy I agree that it is totally okay if we can just use rough
information (in fact it is highly non-trivial to get accurate info without
backend info). My point is that `cur_type_code_` comes from the dtype of
output, but it is totally possible that a compute dag contains computation of
different type code (int8, fp16)
----------------------------------------------------------------
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]
