quic-sanirudh commented on PR #15601: URL: https://github.com/apache/tvm/pull/15601#issuecomment-1694707948
> Approved after discussion. Long-term, the `PassContext` constructed here should probably take all options other than `pass_config` from the current pass context. Sure, I agree that in the long term, it might be better to pick it up from PassContext. Perhaps to avoid a breaking change initially, we can keep the arguments and prioritize them when provided, but when they're not provided, they can be picked up from the current PassContext. -- 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]
