Skip to content

[build] Move a number of ATen checks out of Dependencies.cmake#12990

Closed
orionr wants to merge 3 commits intopytorch:masterfrom
orionr:cmake-cleanup
Closed

[build] Move a number of ATen checks out of Dependencies.cmake#12990
orionr wants to merge 3 commits intopytorch:masterfrom
orionr:cmake-cleanup

Conversation

@orionr
Copy link
Copy Markdown
Contributor

@orionr orionr commented Oct 23, 2018

Comment thread CMakeLists.txt Outdated

This comment was marked as off-topic.

This comment was marked as off-topic.

@Yangqing
Copy link
Copy Markdown
Contributor

Thanks! One nit commit and the rest looks good.

@orionr
Copy link
Copy Markdown
Contributor Author

orionr commented Oct 25, 2018

Build timeouts on a few, but otherwise looks good.

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

orionr is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

zdevito pushed a commit to zdevito/ATen that referenced this pull request Oct 26, 2018
Summary:
cc Yangqing mingzhe09088 anderspapitto mingzhe09088
Pull Request resolved: pytorch/pytorch#12990

Differential Revision: D10862301

Pulled By: orionr

fbshipit-source-id: 62ba09cf0725f29692fac71bc30173469283390b
@ezyang ezyang added the merged label Jun 25, 2019
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
cc Yangqing mingzhe09088 anderspapitto mingzhe09088
Pull Request resolved: pytorch#12990

Differential Revision: D10862301

Pulled By: orionr

fbshipit-source-id: 62ba09cf0725f29692fac71bc30173469283390b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants