Skip to content

make the predicate truly data dependent. #4130

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

Closed
wants to merge 1 commit into from

Conversation

ydwu4
Copy link
Contributor

@ydwu4 ydwu4 commented Jul 2, 2024

Summary: Make the pred truely data-dependent to avoid specializing into one of the branches if we land pytorch/pytorch#128709.

Differential Revision: D59287189

Copy link

pytorch-bot bot commented Jul 2, 2024

🔗 Helpful Links

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

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

✅ No Failures

As of commit 6d38493 with merge base b91c20b (image):
💚 Looks good so far! There are no failures yet. 💚

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

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 2, 2024
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D59287189

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D59287189

@ydwu4 ydwu4 force-pushed the export-D59287189 branch from affcdfe to 2d2434f Compare July 8, 2024 18:42
ydwu4 added a commit to ydwu4/executorch-1 that referenced this pull request Jul 8, 2024
Summary:
Pull Request resolved: pytorch#4130

Make the pred truely data-dependent to avoid specializing into one of the branches if we land   pytorch/pytorch#128709.

Reviewed By: angelayi

Differential Revision: D59287189
Summary:
Pull Request resolved: pytorch#4130

Make the pred truely data-dependent to avoid specializing into one of the branches if we land   pytorch/pytorch#128709.

Reviewed By: angelayi

Differential Revision: D59287189
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D59287189

@ydwu4 ydwu4 force-pushed the export-D59287189 branch from 2d2434f to 6d38493 Compare July 8, 2024 18:48
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 5f2ab0e.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants