Skip to content

unbreak optimized gelu test #7987

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

Merged
merged 1 commit into from
Jan 28, 2025
Merged

unbreak optimized gelu test #7987

merged 1 commit into from
Jan 28, 2025

Conversation

swolchok
Copy link
Contributor

We don't run optimized gelu in OSS currently (soon, I hope!), so I missed the need to fix it.

Test plan: ran optimized gelu test internally.

[ghstack-poisoned]
@swolchok
Copy link
Contributor Author

swolchok commented Jan 27, 2025

Stack from ghstack (oldest at bottom):

Copy link

pytorch-bot bot commented Jan 27, 2025

🔗 Helpful Links

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

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

⏳ No Failures, 36 Pending

As of commit baccacd with merge base da0d85d (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 Jan 27, 2025
swolchok added a commit that referenced this pull request Jan 27, 2025
We don't run optimized gelu in OSS currently (soon, I hope!), so I missed the need to fix it.

Test plan: ran optimized gelu test internally.

ghstack-source-id: 7663626
ghstack-comment-id: 2617218239
Pull Request resolved: #7987
@swolchok
Copy link
Contributor Author

merging despite unfinished mac unittest because I ran those locally

@swolchok swolchok merged commit 019df69 into main Jan 28, 2025
43 of 46 checks passed
@swolchok swolchok deleted the gh/swolchok/211/head branch January 28, 2025 00:15
swolchok added a commit that referenced this pull request Jan 28, 2025
#7987)

The code gated under this `if` does disallowed static_casts if the `if` condition doesn't pass; this is what `if constexpr` is for.

ghstack-source-id: 8286f21
ghstack-comment-id: 2620014279
Pull Request resolved: #8011
swolchok added a commit that referenced this pull request Jan 28, 2025
#7987) (#8011)

The code gated under this `if` does disallowed `static_cast`s if the `if` condition doesn't pass; this is what `if constexpr` is for.
YIWENX14 pushed a commit that referenced this pull request Jan 28, 2025
We don't run optimized gelu in OSS currently (soon, I hope!), so I missed the need to fix it.

Test plan: ran optimized gelu test internally.
YIWENX14 pushed a commit that referenced this pull request Jan 28, 2025
#7987) (#8011)

The code gated under this `if` does disallowed `static_cast`s if the `if` condition doesn't pass; this is what `if constexpr` is for.
zonglinpeng pushed a commit to zonglinpeng/executorch that referenced this pull request Jan 30, 2025
We don't run optimized gelu in OSS currently (soon, I hope!), so I missed the need to fix it.

Test plan: ran optimized gelu test internally.
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. topic: not user facing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants