-
Notifications
You must be signed in to change notification settings - Fork 26.3k
[ATEN->C10] Migrate return type void to () for native functions. #28290
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
Differential Revision: [D17565528](https://our.internmc.facebook.com/intern/diff/D17565528/) [ghstack-poisoned]
Differential Revision: [D17565528](https://our.internmc.facebook.com/intern/diff/D17565528/) ghstack-source-id: 91245072 Pull Request resolved: #28290
…tions." Differential Revision: [D17565528](https://our.internmc.facebook.com/intern/diff/D17565528/) [ghstack-poisoned]
…tions." Differential Revision: [D17565528](https://our.internmc.facebook.com/intern/diff/D17565528/) [ghstack-poisoned]
Pull Request resolved: #28290 ghstack-source-id: 92258422 Differential Revision: [D17565528](https://our.internmc.facebook.com/intern/diff/D17565528/)
…tions." Differential Revision: [D17565528](https://our.internmc.facebook.com/intern/diff/D17565528/) [ghstack-poisoned]
Pull Request resolved: #28290 ghstack-source-id: 92269400 Differential Revision: [D17565528](https://our.internmc.facebook.com/intern/diff/D17565528/)
…tions." Differential Revision: [D17565528](https://our.internmc.facebook.com/intern/diff/D17565528/) [ghstack-poisoned]
Pull Request resolved: #28290 ghstack-source-id: 92272727 Differential Revision: [D17565528](https://our.internmc.facebook.com/intern/diff/D17565528/)
…tions." Differential Revision: [D17565528](https://our.internmc.facebook.com/intern/diff/D17565528/) [ghstack-poisoned]
Pull Request resolved: #28290 ghstack-source-id: 92368250 Differential Revision: [D17565528](https://our.internmc.facebook.com/intern/diff/D17565528/)
smessmer
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, thanks :)
Summary: Pull Request resolved: pytorch/pytorch#28290 ghstack-source-id: 92368250 Test Plan: Pull Request resolved: pytorch/pytorch#28290 ghstack-source-id: 92368250 Differential Revision: D17565528 fbshipit-source-id: f4870bb9ee4f4e7c48df4d68508b512d25ed277c
|
This pull request has been merged in 177c95e. |
|
Why is there no motivation provided for this change? CC @smessmer. |
|
|
Summary: Pull Request resolved: pytorch#28290 ghstack-source-id: 92368250 Test Plan: Pull Request resolved: pytorch#28290 ghstack-source-id: 92368250 Differential Revision: D17565528 fbshipit-source-id: f4870bb9ee4f4e7c48df4d68508b512d25ed277c
Stack from ghstack:
Differential Revision: D17565528