Skip to content

Conversation

@jamesr66a
Copy link
Collaborator

@jamesr66a jamesr66a commented Jun 19, 2019

Stack from ghstack:

This may possibly fix the windows CUDA build breakage introduced by the previous PRs

Differential Revision: D15907633

@pytorchbot pytorchbot added the module: internals Related to internal abstractions in c10 and ATen label Jun 19, 2019
@jamesr66a jamesr66a requested a review from ezyang June 19, 2019 20:33
Copy link
Contributor

@ezyang ezyang left a comment

Choose a reason for hiding this comment

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

you should say why you're moving them into a cpp file in the pr body ;) I'm dumb and can't read

@jamesr66a
Copy link
Collaborator Author

Windows build succeeded. I think we're good to go here

@zou3519 zou3519 deleted the gh/jamesr66a/8/head branch June 19, 2019 23:45
zdevito pushed a commit to zdevito/ATen that referenced this pull request Jun 20, 2019
Summary:
Pull Request resolved: pytorch/pytorch#21983
ghimport-source-id: fe9c1eba5f4c737e1442b877d396b9e8e5298cfb

Test Plan: Imported from OSS

Reviewed By: suo

Differential Revision: D15907633

Pulled By: jamesr66a

fbshipit-source-id: bd2dfdca117cdc3ae35fdd9d29cf521d82636069
@facebook-github-bot
Copy link
Contributor

@jamesr66a merged this pull request in 15be548.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged module: internals Related to internal abstractions in c10 and ATen

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants