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

   Prior to this commit, the `relax.transform.RemoveUnusedOutputs` pass only 
marked a tuple element as used if it occurred in a `TupleGetItem` node.  This 
ignored use cases where a tuple is used as an aggregate object, such as 
returning a tuple from a function.  This would collect incorrect results for a 
Relax function that calls a subroutine, receives a tuple as the return value of 
the subroutine, then returns that tuple.
   
   This commit updates `RemoveUnusedOutputs` to look for usage of a tuple 
object, not just for usage in `TupleGetItem`.
   
   Closes https://github.com/apache/tvm/issues/17247


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