Skip to content

[executorch] Optimized 2D-by-1D broadcasting in optimized op_mul #4808

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 4 commits into from
Aug 28, 2024

Conversation

swolchok
Copy link
Contributor

@swolchok swolchok commented Aug 20, 2024

Detect that we are doing an elementwise multiplication for a 2D tensor and a 1D tensor. Dispatch to a vectorized kernel for this case.

Differential Revision: [D61560826](https://our.internmc.facebook.com/intern/diff/D61560826/)

[ghstack-poisoned]
Copy link

pytorch-bot bot commented Aug 20, 2024

🔗 Helpful Links

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

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

✅ You can merge normally! (1 Unrelated Failure)

As of commit fefc4a2 with merge base 35e2302 (image):

FLAKY - The following job failed but was likely due to flakiness present on trunk:

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 Aug 20, 2024
swolchok added a commit that referenced this pull request Aug 20, 2024
Detect that we are doing an elementwise multiplication for a 2D tensor and a 1D tensor. Dispatch to a vectorized kernel for this case.

Differential Revision: [D61560826](https://our.internmc.facebook.com/intern/diff/D61560826/)

ghstack-source-id: 238994618
Pull Request resolved: #4808
@facebook-github-bot
Copy link
Contributor

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

@swolchok swolchok requested review from kimishpatel, GregoryComer and manuelcandales and removed request for kimishpatel August 20, 2024 21:22
…op_mul"

Detect that we are doing an elementwise multiplication for a 2D tensor and a 1D tensor. Dispatch to a vectorized kernel for this case.

Differential Revision: [D61560826](https://our.internmc.facebook.com/intern/diff/D61560826/)

[ghstack-poisoned]
@facebook-github-bot
Copy link
Contributor

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

swolchok added a commit that referenced this pull request Aug 20, 2024
Pull Request resolved: #4808

Detect that we are doing an elementwise multiplication for a 2D tensor and a 1D tensor. Dispatch to a vectorized kernel for this case.
ghstack-source-id: 239001636
@exported-using-ghexport

Differential Revision: [D61560826](https://our.internmc.facebook.com/intern/diff/D61560826/)
…op_mul"

Detect that we are doing an elementwise multiplication for a 2D tensor and a 1D tensor. Dispatch to a vectorized kernel for this case.

Differential Revision: [D61560826](https://our.internmc.facebook.com/intern/diff/D61560826/)

[ghstack-poisoned]
@facebook-github-bot
Copy link
Contributor

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

…op_mul"

Detect that we are doing an elementwise multiplication for a 2D tensor and a 1D tensor. Dispatch to a vectorized kernel for this case.

Differential Revision: [D61560826](https://our.internmc.facebook.com/intern/diff/D61560826/)

[ghstack-poisoned]
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot facebook-github-bot merged commit 0e077cf into gh/swolchok/18/base Aug 28, 2024
36 of 38 checks passed
@facebook-github-bot facebook-github-bot deleted the gh/swolchok/18/head branch August 28, 2024 21:59
kirklandsign added a commit that referenced this pull request Aug 29, 2024
Detect that we are doing an elementwise multiplication for a 2D tensor and a 1D tensor. Dispatch to a vectorized kernel for this case.

Differential Revision: [D61560826](https://our.internmc.facebook.com/intern/diff/D61560826/)

Pull Request resolved: #4808

---------

Co-authored-by: Scott Wolchok <[email protected]>
kedarnath03 pushed a commit to kedarnath03/executorch that referenced this pull request Jun 25, 2025
Pull Request resolved: pytorch/executorch#4808

Detect that we are doing an elementwise multiplication for a 2D tensor and a 1D tensor. Dispatch to a vectorized kernel for this case.
ghstack-source-id: 240122842
@exported-using-ghexport

Differential Revision: [D61560826](https://our.internmc.facebook.com/intern/diff/D61560826/)
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants