Skip to content

[ExecuTorch] Don't require -march compiler flags to use bfdot#5444

Closed
swolchok wants to merge 3 commits intogh/swolchok/75/basefrom
gh/swolchok/75/head
Closed

[ExecuTorch] Don't require -march compiler flags to use bfdot#5444
swolchok wants to merge 3 commits intogh/swolchok/75/basefrom
gh/swolchok/75/head

Conversation

@swolchok
Copy link
Contributor

@swolchok swolchok commented Sep 17, 2024

Stack from ghstack (oldest at bottom):

TIL about the target clang/GCC function attribute, which allows building a particular function under an -march flag instead of a whole file.

Differential Revision: D62905047

TIL about the `target` clang/GCC function attribute, which allows building a particular function under an `-march` flag instead of a whole file.

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

[ghstack-poisoned]
@pytorch-bot
Copy link

pytorch-bot bot commented Sep 17, 2024

🔗 Helpful Links

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

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

✅ No Failures

As of commit b13a7ee with merge base 3512148 (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 Sep 17, 2024
@facebook-github-bot
Copy link
Contributor

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

swolchok added a commit that referenced this pull request Sep 17, 2024
TIL about the `target` clang/GCC function attribute, which allows building a particular function under an `-march` flag instead of a whole file.

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

ghstack-source-id: 243159890
Pull Request resolved: #5444
…dot"

TIL about the `target` clang/GCC function attribute, which allows building a particular function under an `-march` flag instead of a whole file.

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

[ghstack-poisoned]
swolchok added a commit that referenced this pull request Sep 19, 2024
Pull Request resolved: #5444

TIL about the `target` clang/GCC function attribute, which allows building a particular function under an `-march` flag instead of a whole file.

Differential Revision: [D62905047](https://our.internmc.facebook.com/intern/diff/D62905047/)
ghstack-source-id: 243534339
@facebook-github-bot
Copy link
Contributor

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

swolchok added a commit to pytorch/pytorch that referenced this pull request Sep 19, 2024
ExecuTorch's fork of BlasKernel.cpp grew bfdot support, complete with demonstration that it helps. Port it back to PyTorch. Supersedes #127488 . Includes not-yet-reviewed pytorch/executorch#5444 .

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

[ghstack-poisoned]
swolchok added a commit to pytorch/pytorch that referenced this pull request Sep 19, 2024
ExecuTorch's fork of BlasKernel.cpp grew bfdot support, complete with demonstration that it helps. Port it back to PyTorch. Supersedes #127488 . Includes not-yet-reviewed pytorch/executorch#5444 .

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

ghstack-source-id: 243596406
Pull Request resolved: #136331
…dot"

TIL about the `target` clang/GCC function attribute, which allows building a particular function under an `-march` flag instead of a whole file.

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

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

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

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

TIL about the `target` clang/GCC function attribute, which allows building a particular function under an `-march` flag instead of a whole file.
ghstack-source-id: 243858419

Differential Revision: [D62905047](https://our.internmc.facebook.com/intern/diff/D62905047/)
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 0eee42a.

swolchok added a commit to pytorch/pytorch that referenced this pull request Sep 23, 2024
…t back to ATen BlasKernel"

ExecuTorch's fork of BlasKernel.cpp grew bfdot support, complete with demonstration that it helps. Port it back to PyTorch. Supersedes #127488 . Includes not-yet-reviewed pytorch/executorch#5444 .

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

[ghstack-poisoned]
swolchok added a commit to pytorch/pytorch that referenced this pull request Sep 23, 2024
…lasKernel"

ExecuTorch's fork of BlasKernel.cpp grew bfdot support, complete with demonstration that it helps. Port it back to PyTorch. Supersedes #127488 . Includes not-yet-reviewed pytorch/executorch#5444 .

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

[ghstack-poisoned]
swolchok added a commit to pytorch/pytorch that referenced this pull request Sep 23, 2024
…t back to ATen BlasKernel"

ExecuTorch's fork of BlasKernel.cpp grew bfdot support, complete with demonstration that it helps. Port it back to PyTorch. Supersedes #127488 . Includes not-yet-reviewed pytorch/executorch#5444 .

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

[ghstack-poisoned]
swolchok added a commit to pytorch/pytorch that referenced this pull request Sep 23, 2024
…lasKernel"

ExecuTorch's fork of BlasKernel.cpp grew bfdot support, complete with demonstration that it helps. Port it back to PyTorch. Supersedes #127488 . Includes not-yet-reviewed pytorch/executorch#5444 .

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

[ghstack-poisoned]
swolchok added a commit to pytorch/pytorch that referenced this pull request Sep 23, 2024
Pull Request resolved: #136331

ExecuTorch's fork of BlasKernel.cpp grew bfdot support, complete with demonstration that it helps. Port it back to PyTorch. Supersedes #127488 . Includes not-yet-reviewed pytorch/executorch#5444 .
ghstack-source-id: 244187352

Differential Revision: [D63045939](https://our.internmc.facebook.com/intern/diff/D63045939/)
swolchok added a commit to pytorch/pytorch that referenced this pull request Sep 30, 2024
Pull Request resolved: #136331

ExecuTorch's fork of BlasKernel.cpp grew bfdot support, complete with demonstration that it helps. Port it back to PyTorch. Supersedes #127488 . Includes pytorch/executorch#5444 .
ghstack-source-id: 245455630

Differential Revision: [D63045939](https://our.internmc.facebook.com/intern/diff/D63045939/)
swolchok added a commit to pytorch/pytorch that referenced this pull request Sep 30, 2024
…t back to ATen BlasKernel"


ExecuTorch's fork of BlasKernel.cpp grew bfdot support, complete with demonstration that it helps. Port it back to PyTorch. Supersedes #127488 . Includes pytorch/executorch#5444 .

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

[ghstack-poisoned]
swolchok added a commit to pytorch/pytorch that referenced this pull request Sep 30, 2024
…lasKernel"


ExecuTorch's fork of BlasKernel.cpp grew bfdot support, complete with demonstration that it helps. Port it back to PyTorch. Supersedes #127488 . Includes pytorch/executorch#5444 .

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

[ghstack-poisoned]
swolchok added a commit to pytorch/pytorch that referenced this pull request Oct 2, 2024
Pull Request resolved: #136331

ExecuTorch's fork of BlasKernel.cpp grew bfdot support, complete with demonstration that it helps. Port it back to PyTorch. Supersedes #127488 . Includes pytorch/executorch#5444 .
ghstack-source-id: 245903000

Differential Revision: [D63045939](https://our.internmc.facebook.com/intern/diff/D63045939/)
swolchok added a commit to pytorch/pytorch that referenced this pull request Oct 2, 2024
…t back to ATen BlasKernel"


ExecuTorch's fork of BlasKernel.cpp grew bfdot support, complete with demonstration that it helps. Port it back to PyTorch. Supersedes #127488 . Includes pytorch/executorch#5444 .

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

[ghstack-poisoned]
swolchok added a commit to pytorch/pytorch that referenced this pull request Oct 2, 2024
…lasKernel"


ExecuTorch's fork of BlasKernel.cpp grew bfdot support, complete with demonstration that it helps. Port it back to PyTorch. Supersedes #127488 . Includes pytorch/executorch#5444 .

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

[ghstack-poisoned]
pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request Oct 3, 2024
…136331)

ExecuTorch's fork of BlasKernel.cpp grew bfdot support, complete with demonstration that it helps. Port it back to PyTorch. Supersedes #127488 . Includes pytorch/executorch#5444 .

Differential Revision: [D63045939](https://our.internmc.facebook.com/intern/diff/D63045939/)
Pull Request resolved: #136331
Approved by: https://github.com/malfet, https://github.com/albanD
ghstack dependencies: #136445
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