-
Notifications
You must be signed in to change notification settings - Fork 21.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[inductor] Kill mark_node_as_mutating #129346
Conversation
ghstack-source-id: 8f9ac1f098507425d31f91e8ba5d88174b7fabc6 Pull Request resolved: #129346
ghstack-source-id: d1d1c4fb8ed2879786ab3ba878d193a555eb631b Pull Request resolved: #129346
ghstack-source-id: 952fb9b61d2889d58cd1162f1925c6d7cc795dd4 Pull Request resolved: #129346
ghstack-source-id: 971dcc1c51a405de107cfade81b4dc7de21e2f32 Pull Request resolved: #129346
@ColinPeppler thank you for the reproducer and fix. Should be fixed now in #128893 @yf225 I think this is worth doing as we need to support operations with multiple outputs (including mutations) and up till this point this was done by a variety of hacky workarounds. e.g. |
@ColinPeppler would you mind importing this PR to see if the errors are fixed? |
|
ghstack-source-id: fb6ddbc1d593058ea109057a40e7d70d9706c2fc Pull Request resolved: pytorch#129346
Resubmit of pytorch#129346 ghstack-source-id: fe1b4aca578876db67e0fbb6331ca2afd6e0c990 Pull Request resolved: pytorch#130834
Resubmit of pytorch#129346 Pull Request resolved: pytorch#130834 Approved by: https://github.com/lezcano ghstack dependencies: pytorch#130831, pytorch#130832, pytorch#130833
Resubmit of pytorch#129346 ghstack-source-id: 498a08bb82c07900c9d9f0d686a7ec6d727ec114 Pull Request resolved: pytorch#130834
Resubmit of pytorch#129346 ghstack-source-id: 0d8e6a94dc0352b71f85dd43871985cfaca55aac Pull Request resolved: pytorch#130834
Resubmit of #129346 Pull Request resolved: #130834 Approved by: https://github.com/lezcano ghstack dependencies: #130832, #130833
Summary: Revert pytorch#129346 thru pytorch#128893 For S430832 Test Plan: Tests Differential Revision: D59503843 Pull Request resolved: pytorch#130303 Approved by: https://github.com/bdhirsh
Resubmit of pytorch#129346 Pull Request resolved: pytorch#130834 Approved by: https://github.com/lezcano ghstack dependencies: pytorch#130831, pytorch#130832, pytorch#130833
Resubmit of pytorch#129346 Pull Request resolved: pytorch#130834 Approved by: https://github.com/lezcano ghstack dependencies: pytorch#130832, pytorch#130833
Stack from ghstack (oldest at bottom):
cc @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @ipiszy @yf225 @chenyang78 @kadeng @muchulee8 @ColinPeppler @amjames @desertfire @chauhang