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

   Prior to this commit, the `relax.transform.LiftTransformParams` pass 
inspected the expression in each `relax::Binding` for variables that were 
required at runtime, but did not inspect the function's output. As a result, 
any value that could be computed at compile-time, and was either the function 
output or used in the function's output tuple, would be undefined in the 
inference function.
   
   This commit updates `LiftTransformParams` to collect variables from both the 
bound value of `relax::Binding`, and the function's output.
   
   While this error only impacted the `shared_transform=False` branch of 
`LiftTransformParams`, this commit also adds regression tests the 
`shared_transform=True` use case of `LiftTransformParams`.
   
   Closes https://github.com/apache/tvm/issues/17231


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

Reply via email to