Skip to content

Conversation

@xcheng16
Copy link
Contributor

@xcheng16 xcheng16 commented Oct 18, 2019

Stack from ghstack:

Differential Revision: D17565528

@facebook-github-bot facebook-github-bot added the oncall: jit Add this issue/PR to JIT oncall triage queue label Oct 18, 2019
xcheng16 pushed a commit that referenced this pull request Oct 18, 2019
Differential Revision: [D17565528](https://our.internmc.facebook.com/intern/diff/D17565528/)

ghstack-source-id: 91245072
Pull Request resolved: #28290
xcheng16 pushed a commit that referenced this pull request Oct 19, 2019
Pull Request resolved: #28290


ghstack-source-id: 92258422

Differential Revision: [D17565528](https://our.internmc.facebook.com/intern/diff/D17565528/)
xcheng16 pushed a commit that referenced this pull request Oct 21, 2019
Pull Request resolved: #28290


ghstack-source-id: 92269400

Differential Revision: [D17565528](https://our.internmc.facebook.com/intern/diff/D17565528/)
xcheng16 pushed a commit that referenced this pull request Oct 21, 2019
Pull Request resolved: #28290


ghstack-source-id: 92272727

Differential Revision: [D17565528](https://our.internmc.facebook.com/intern/diff/D17565528/)
@xcheng16 xcheng16 requested review from dreiss and smessmer October 21, 2019 16:31
xcheng16 pushed a commit that referenced this pull request Oct 22, 2019
Pull Request resolved: #28290


ghstack-source-id: 92368250

Differential Revision: [D17565528](https://our.internmc.facebook.com/intern/diff/D17565528/)
Copy link
Contributor

@smessmer smessmer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, thanks :)

zdevito pushed a commit to zdevito/ATen that referenced this pull request Oct 22, 2019
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
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 177c95e.

@facebook-github-bot facebook-github-bot deleted the gh/xcheng16/3/head branch October 28, 2019 22:21
@gchanan
Copy link
Contributor

gchanan commented Oct 31, 2019

Why is there no motivation provided for this change? CC @smessmer.

@smessmer
Copy link
Contributor

smessmer commented Nov 1, 2019

() is how the JIT function schema represents void returns. This is part of the effort of aligning function schemas and was required for migrating operators to c10.

thiagocrepaldi pushed a commit to thiagocrepaldi/pytorch that referenced this pull request Feb 4, 2020
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged oncall: jit Add this issue/PR to JIT oncall triage queue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants