Skip to content

When BUILD_ATEN=OFF, use ATen/core directly#10019

Closed
ezyang wants to merge 10 commits intopytorch:masterfrom
ezyang:pr/when-no-build-aten-use-aten-core-directly
Closed

When BUILD_ATEN=OFF, use ATen/core directly#10019
ezyang wants to merge 10 commits intopytorch:masterfrom
ezyang:pr/when-no-build-aten-use-aten-core-directly

Conversation

@ezyang
Copy link
Contributor

@ezyang ezyang commented Jul 30, 2018

ATenCore.h is a dummy header to just test that this is working at all.

ezyang added 7 commits July 30, 2018 11:31
Signed-off-by: Edward Z. Yang <ezyang@fb.com>
Signed-off-by: Edward Z. Yang <ezyang@fb.com>
Signed-off-by: Edward Z. Yang <ezyang@fb.com>
Signed-off-by: Edward Z. Yang <ezyang@fb.com>
Signed-off-by: Edward Z. Yang <ezyang@fb.com>
Signed-off-by: Edward Z. Yang <ezyang@fb.com>
Signed-off-by: Edward Z. Yang <ezyang@fb.com>
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.

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

Copy link
Contributor

@orionr orionr left a comment

Choose a reason for hiding this comment

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

Looks like a good start!

add_subdirectory(../aten/src/ATen/core ATen_core)
list(APPEND Caffe2_CPU_SRCS ${ATen_CORE_SRCS})
list(APPEND Caffe2_CPU_INCLUDE ${ATen_CORE_INCLUDE})
# TODO: We should probably install the headers, but I don't know

This comment was marked as off-topic.

# NB: If you edit these globs, you'll have to update setup.py package_data as well
FILE(GLOB base_h "*.h" "detail/*.h")
FILE(GLOB base_cpp "*.cpp" "detail/*.cpp")
add_subdirectory(core)

This comment was marked as off-topic.

ezyang added 3 commits July 30, 2018 12:26
Signed-off-by: Edward Z. Yang <ezyang@fb.com>
Signed-off-by: Edward Z. Yang <ezyang@fb.com>
Signed-off-by: Edward Z. Yang <ezyang@fb.com>
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.

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

facebook-github-bot pushed a commit that referenced this pull request Jul 30, 2018
Summary:
Extracted from 6276246 and in support of #10019

cc pjh5 mingzhe09088 ezyang smessmer
Pull Request resolved: #10020

Reviewed By: pjh5

Differential Revision: D9068124

Pulled By: orionr

fbshipit-source-id: 4dd4910136a312b6517c65ce8802837108475f89
zdevito pushed a commit to zdevito/ATen that referenced this pull request Jul 31, 2018
Summary:
ATenCore.h is a dummy header to just test that this is working at all.
Pull Request resolved: pytorch/pytorch#10019

Reviewed By: smessmer

Differential Revision: D9067262

Pulled By: ezyang

fbshipit-source-id: 58bab9c0aa83b56335e36b719b9b6505400d8dee
jramseyer pushed a commit to jramseyer/pytorch that referenced this pull request Jul 31, 2018
Summary:
Extracted from pytorch@6276246 and in support of pytorch#10019

cc pjh5 mingzhe09088 ezyang smessmer
Pull Request resolved: pytorch#10020

Reviewed By: pjh5

Differential Revision: D9068124

Pulled By: orionr

fbshipit-source-id: 4dd4910136a312b6517c65ce8802837108475f89
jramseyer pushed a commit to jramseyer/pytorch that referenced this pull request Jul 31, 2018
Summary:
ATenCore.h is a dummy header to just test that this is working at all.
Pull Request resolved: pytorch#10019

Reviewed By: smessmer

Differential Revision: D9067262

Pulled By: ezyang

fbshipit-source-id: 58bab9c0aa83b56335e36b719b9b6505400d8dee
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary:
Extracted from pytorch@6276246 and in support of pytorch#10019

cc pjh5 mingzhe09088 ezyang smessmer
Pull Request resolved: pytorch#10020

Reviewed By: pjh5

Differential Revision: D9068124

Pulled By: orionr

fbshipit-source-id: 4dd4910136a312b6517c65ce8802837108475f89
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary:
ATenCore.h is a dummy header to just test that this is working at all.
Pull Request resolved: pytorch#10019

Reviewed By: smessmer

Differential Revision: D9067262

Pulled By: ezyang

fbshipit-source-id: 58bab9c0aa83b56335e36b719b9b6505400d8dee
@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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants