-
Notifications
You must be signed in to change notification settings - Fork 21.5k
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
[AOTI] Disable stack allocation for OSS #125732
Conversation
Summary: Stack allocation is for certain small CPU models, but its coverage still needs improvement, so default to OFF for OSS. [ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/125732
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 788e458 with merge base b40fb2d (): UNSTABLE - The following job failed but was likely due to flakiness present on trunk and has been marked as unstable:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
Summary: Stack allocation is for certain small CPU models, but its coverage still needs improvement, so default to OFF for OSS. [ghstack-poisoned]
Summary: Stack allocation is for certain small CPU models, but its coverage still needs improvement, so default to OFF for OSS. ghstack-source-id: 51693eb90ba97406a0e35689809196ffc8a73768 Pull Request resolved: #125732
Summary: Stack allocation is for certain small CPU models, but its coverage still needs improvement, so default to OFF for OSS. cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx peterbell10 ipiszy yf225 chenyang78 kadeng muchulee8 ColinPeppler amjames chauhang [ghstack-poisoned]
Summary: Stack allocation is for certain small CPU models, but its coverage still needs improvement, so default to OFF for OSS. cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx peterbell10 ipiszy yf225 chenyang78 kadeng muchulee8 ColinPeppler amjames chauhang [ghstack-poisoned]
Summary: Stack allocation is for certain small CPU models, but its coverage still needs improvement, so default to OFF for OSS. cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx peterbell10 ipiszy yf225 chenyang78 kadeng muchulee8 ColinPeppler amjames chauhang [ghstack-poisoned]
Summary: Stack allocation is for certain small CPU models, but its coverage still needs improvement, so default to OFF for OSS. cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx peterbell10 ipiszy yf225 chenyang78 kadeng muchulee8 ColinPeppler amjames chauhang [ghstack-poisoned]
@pytorchbot merge |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
This reverts commit 599e684. Reverted #125732 on behalf of https://github.com/DanilBaibak due to Break internal build ([comment](#126720 (comment)))
@desertfire your PR has been successfully reverted. |
Summary: Stack allocation is for certain small CPU models, but its coverage still needs improvement, so default to OFF for OSS. cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx peterbell10 ipiszy yf225 chenyang78 kadeng muchulee8 ColinPeppler amjames chauhang [ghstack-poisoned]
Summary: Stack allocation is for certain small CPU models, but its coverage still needs improvement, so default to OFF for OSS. ghstack-source-id: c0c6ccbdb50ecd5c0dfea4f1573c81b433862af5 Pull Request resolved: #125732
@pytorchbot merge |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
This reverts commit 599e684. Reverted pytorch#125732 on behalf of https://github.com/DanilBaibak due to Break internal build ([comment](pytorch#126720 (comment)))
Summary: Stack allocation is for certain small CPU models, but its coverage still needs improvement, so default to OFF for OSS. Pull Request resolved: pytorch#125732 Approved by: https://github.com/chenyang78 ghstack dependencies: pytorch#126720, pytorch#126801
Stack from ghstack (oldest at bottom):
Summary: Stack allocation is for certain small CPU models, but its coverage still needs improvement, so default to OFF for OSS.
cc @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @peterbell10 @ipiszy @yf225 @chenyang78 @kadeng @muchulee8 @ColinPeppler @amjames @chauhang