Lunderberg opened a new pull request, #15688:
URL: https://github.com/apache/tvm/pull/15688

   Prior to this commit, writing a subclass of `relax::ExprVisitor` or 
`relax::ExprMutator` required separate overrides for visiting a 
`relax::DataflowVar` and a `relax::Var`.  In the majority of cases, these two 
types should be treated identically, and failure to handle a `DataflowVar` 
equivalently would be a bug.
   
   This commit updates the `relax::ExprVisitor` and `relax::ExprMutator` base 
classes to visit `DataflowVar` by delegate to the visitor of `relax::Var`.  As 
a result, any derived class that overrides the `relax::Var` visitor will also 
update the behavior for `relax::DataflowVar`.  A pass that requires different 
behavior for `relax::Var` and `relax::DataflowVar` can still explicitly 
override both methods in order to provide different behavior.


-- 
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]

Reply via email to