-
Notifications
You must be signed in to change notification settings - Fork 370
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
Add pass to merge consecutive view ops #768
Conversation
@int3 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
6868284
to
284652e
Compare
@int3 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@int3 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
3 similar comments
@int3 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@int3 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@int3 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@int3 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@int3 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Summary: A sequence of view ops may break many of our transformations, where we assume a single view op. Pull Request resolved: facebookincubator#768 Differential Revision: D46696966 Pulled By: int3 fbshipit-source-id: 92bdda891c2feb53821d51b8b6808e3d3544330c
@int3 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
A sequence of view ops may break many of our transformations, where we assume a single view op.
@int3 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
A sequence of view ops may break many of our transformations, where we assume a single view op.