haoyang9804 commented on PR #15386: URL: https://github.com/apache/tvm/pull/15386#issuecomment-1702692212
> From one hand LGTM, from another one it confuses me due to fix for specific op was done on very high level (and CI test was constructed for specific op not for all cases). It looks like it has been assumed that outputs exist at any `ret_name in ret_names`. I need time to analyze it by my-self or explanation what is wrong with the initial assumption. I agree that "fix for specific op was done on very high level". So I make the fix only specific to the buggy pytorch op. This is only a workaround currently. I believe there are some deeper issues make this bug happen. -- 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]
