electriclilies commented on a change in pull request #9569:
URL: https://github.com/apache/tvm/pull/9569#discussion_r755603629
##########
File path: src/relay/ir/expr_functor.cc
##########
@@ -512,9 +464,9 @@ class ExprBinder : public MixedModeMutator, PatternMutator {
Pattern VisitPattern(const Pattern& p) final { return
PatternMutator::VisitPattern(p); }
- Clause VisitClause(const Clause& c) final {
- Pattern pat = VisitPattern(c->lhs);
- return Clause(pat, VisitExpr(c->rhs));
+ Clause VisitClause(const Clause& clause) final {
+ Pattern lhs = VisitPattern(clause->lhs);
+ return WithFields(std::move(clause), std::move(lhs),
VisitExpr(clause->rhs));
Review comment:
Oh, didn't even notice it was const. I was just std::moving everything
that was named that we were not using again
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
