-
Notifications
You must be signed in to change notification settings - Fork 25.6k
[AOTI] Add aoti_torch_view_dtype in C shim #118705
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
Conversation
Summary: Support ir.ComplexView in the ABI-compatible codegen [ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/118705
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 742b8f8 with merge base abe3c55 ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
@desertfire has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Summary: Support ir.ComplexView in the ABI-compatible codegen cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx peterbell10 ipiszy yf225 chenyang78 kadeng muchulee8 aakhundov ColinPeppler Differential Revision: [D53249039](https://our.internmc.facebook.com/intern/diff/D53249039) [ghstack-poisoned]
@desertfire has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
1 similar comment
@desertfire has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@desertfire has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@pytorchbot merge -f 'Landed internally' (Initiating merge automatically since Phabricator Diff has merged, using force because this PR might not pass merge_rules.json but landed internally) |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Summary: At some point I will stop manually adding ops to C shim, but use torchgen to generate those code. For the near term, I need to add a few more in order to switch the AOTInductor dashboard run. Differential Revision: [D53249074](https://our.internmc.facebook.com/intern/diff/D53249074) Pull Request resolved: #118706 Approved by: https://github.com/frank-wei, https://github.com/aakhundov ghstack dependencies: #118704, #118705
Summary: Support ir.ComplexView in the ABI-compatible codegen Differential Revision: [D53249039](https://our.internmc.facebook.com/intern/diff/D53249039) Pull Request resolved: #118705 Approved by: https://github.com/frank-wei ghstack dependencies: #118704
Summary: At some point I will stop manually adding ops to C shim, but use torchgen to generate those code. For the near term, I need to add a few more in order to switch the AOTInductor dashboard run. Differential Revision: [D53249074](https://our.internmc.facebook.com/intern/diff/D53249074) Pull Request resolved: #118706 Approved by: https://github.com/frank-wei, https://github.com/aakhundov ghstack dependencies: #118704, #118705
Stack from ghstack (oldest at bottom):
Summary: Support ir.ComplexView in the ABI-compatible codegen
cc @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @peterbell10 @ipiszy @yf225 @chenyang78 @kadeng @muchulee8 @aakhundov @ColinPeppler
Differential Revision: D53249039