Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[pytorch] build torch for libtorch mobile build #21234

Closed
wants to merge 1 commit into from

Conversation

ljk53
Copy link
Contributor

@ljk53 ljk53 commented May 31, 2019

Stack from ghstack:

Summary:
Since recent unification of caffe2 and torch library we stop building
torch for mobile. This diff re-enables torch build for libtorch mobile
build unless BUILD_CAFFE2_MOBILE is set to TRUE.

BUILD_CAFFE2_MOBILE is used to build old libcaffe2 library which will
be deprecated. We can remove these conditions after the deprecation.

Test Plan:
Build mobile library and make sure it can run pytorch model on Android.

Differential Revision: D15616540

Summary:
Since recent unification of caffe2 and torch library we stop building
torch for mobile. This diff re-enables torch build for libtorch mobile
build unless BUILD_CAFFE2_MOBILE is set to TRUE.

BUILD_CAFFE2_MOBILE is used to build old libcaffe2 library which will
be deprecated. We can remove these conditions after the deprecation.

Test Plan:
Build mobile library and make sure it can run pytorch model on Android.
@pytorchbot pytorchbot added caffe2 module: build Build system issues labels May 31, 2019
@ljk53 ljk53 requested review from kostmo, orionr and pjh5 May 31, 2019 23:08
@zou3519 zou3519 deleted the gh/ljk53/18/head branch June 4, 2019 02:53
@facebook-github-bot
Copy link
Contributor

@ljk53 merged this pull request in 702ba3d.

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.

None yet

7 participants