Skip to content
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

Closed
wants to merge 27 commits into from

Conversation

[ghstack-poisoned]
Copy link

pytorch-bot bot commented Jun 24, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/129346

Note: Links to docs will display an error until the docs builds have been completed.

❌ 77 Cancelled Jobs, 3 Unrelated Failures

As of commit 40d6db1 with merge base c549629 (image):

CANCELLED JOBS - The following jobs were cancelled. Please retry:

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

[ghstack-poisoned]
peterbell10 added a commit that referenced this pull request Jun 24, 2024
ghstack-source-id: 8f9ac1f098507425d31f91e8ba5d88174b7fabc6
Pull Request resolved: #129346
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
peterbell10 added a commit that referenced this pull request Jun 24, 2024
ghstack-source-id: d1d1c4fb8ed2879786ab3ba878d193a555eb631b
Pull Request resolved: #129346
[ghstack-poisoned]
peterbell10 added a commit that referenced this pull request Jun 25, 2024
ghstack-source-id: 952fb9b61d2889d58cd1162f1925c6d7cc795dd4
Pull Request resolved: #129346
[ghstack-poisoned]
[ghstack-poisoned]
peterbell10 added a commit that referenced this pull request Jun 26, 2024
ghstack-source-id: 971dcc1c51a405de107cfade81b4dc7de21e2f32
Pull Request resolved: #129346
[ghstack-poisoned]
[ghstack-poisoned]
@peterbell10
Copy link
Collaborator Author

peterbell10 commented Jul 11, 2024

@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. MutationOutput as a separate operation doesn't represent the dependencies correctly and implicitly assumes that the scheduler won't reorder nodes.

[ghstack-poisoned]
[ghstack-poisoned]
@peterbell10
Copy link
Collaborator Author

@ColinPeppler would you mind importing this PR to see if the errors are fixed?

[ghstack-poisoned]
Copy link

linux-foundation-easycla bot commented Jul 16, 2024

CLA Missing ID CLA Not Signed

[ghstack-poisoned]
[ghstack-poisoned]
peterbell10 added a commit to peterbell10/pytorch that referenced this pull request Jul 16, 2024
ghstack-source-id: fb6ddbc1d593058ea109057a40e7d70d9706c2fc
Pull Request resolved: pytorch#129346
[ghstack-poisoned]
peterbell10 added a commit to peterbell10/pytorch that referenced this pull request Jul 19, 2024
Resubmit of pytorch#129346

ghstack-source-id: fe1b4aca578876db67e0fbb6331ca2afd6e0c990
Pull Request resolved: pytorch#130834
pytorchmergebot pushed a commit that referenced this pull request Jul 20, 2024
DiweiSun pushed a commit to DiweiSun/pytorch that referenced this pull request Jul 22, 2024
peterbell10 added a commit to peterbell10/pytorch that referenced this pull request Jul 22, 2024
Resubmit of pytorch#129346

ghstack-source-id: 498a08bb82c07900c9d9f0d686a7ec6d727ec114
Pull Request resolved: pytorch#130834
peterbell10 added a commit to peterbell10/pytorch that referenced this pull request Jul 23, 2024
Resubmit of pytorch#129346

ghstack-source-id: 0d8e6a94dc0352b71f85dd43871985cfaca55aac
Pull Request resolved: pytorch#130834
pytorchmergebot pushed a commit that referenced this pull request Jul 24, 2024
Resubmit of #129346

Pull Request resolved: #130834
Approved by: https://github.com/lezcano
ghstack dependencies: #130832, #130833
xuhancn pushed a commit to xuhancn/pytorch that referenced this pull request Jul 25, 2024
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
xuhancn pushed a commit to xuhancn/pytorch that referenced this pull request Jul 25, 2024
xuhancn pushed a commit to xuhancn/pytorch that referenced this pull request Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants