slyubomirsky commented on code in PR #16594:
URL: https://github.com/apache/tvm/pull/16594#discussion_r1499986277
##########
src/relax/transform/canonicalize_bindings.cc:
##########
@@ -247,7 +247,8 @@ class BindingCanonicalizer : public ExprMutator {
}
Expr VisitExpr_(const VarNode* var) override {
- Var new_var = GetRef<Var>(var);
Review Comment:
I don't think it's necessary to remove, especially since it seems like a
correct change. I'm glad that there wasn't a bug that you encountered because
of it.
--
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-unsubscr...@tvm.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
