Skip to content

Correctly set CAFFE2_DISABLE_NUMA when USE_NUMA=OFF in cmake#10061

Closed
bddppq wants to merge 1 commit intopytorch:masterfrom
bddppq:fix-numa-cmake
Closed

Correctly set CAFFE2_DISABLE_NUMA when USE_NUMA=OFF in cmake#10061
bddppq wants to merge 1 commit intopytorch:masterfrom
bddppq:fix-numa-cmake

Conversation

@bddppq
Copy link
Contributor

@bddppq bddppq commented Jul 31, 2018

previously https://github.com/pytorch/pytorch/blob/master/caffe2/core/numa.cc still gets compiled even when USE_NUMA=OFF

@bddppq bddppq requested a review from dzhulgakov July 31, 2018 08:24
Copy link
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.

bddppq has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@bddppq bddppq requested a review from ilia-cher July 31, 2018 08:26
@fmassa fmassa added the caffe2 label Jul 31, 2018
@bddppq
Copy link
Contributor Author

bddppq commented Jul 31, 2018

@pytorchbot retest this please

@bddppq bddppq requested review from houseroad and pjh5 July 31, 2018 17:13
zdevito pushed a commit to zdevito/ATen that referenced this pull request Jul 31, 2018
Summary:
previously https://github.com/pytorch/pytorch/blob/master/caffe2/core/numa.cc still gets compiled even when USE_NUMA=OFF
Pull Request resolved: pytorch/pytorch#10061

Reviewed By: houseroad

Differential Revision: D9081385

Pulled By: bddppq

fbshipit-source-id: ad28b647e0033727839770b1da0fba341b1b7787
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
…#10061)

Summary:
previously https://github.com/pytorch/pytorch/blob/master/caffe2/core/numa.cc still gets compiled even when USE_NUMA=OFF
Pull Request resolved: pytorch#10061

Reviewed By: houseroad

Differential Revision: D9081385

Pulled By: bddppq

fbshipit-source-id: ad28b647e0033727839770b1da0fba341b1b7787
@ezyang ezyang added the merged label Jun 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants