Skip to content

Build libtorch binary with new ABI #23908

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

Closed
wants to merge 54 commits into from
Closed

Conversation

yf225
Copy link
Contributor

@yf225 yf225 commented Aug 6, 2019

This PR enables building libtorch with new ABI, using gcc 5.4 on Ubuntu 16.04.

Accompanying PR: pytorch/builder#335.

@pytorchbot pytorchbot added the module: ci Related to continuous integration label Aug 6, 2019
@yf225 yf225 force-pushed the cpp_new_abi branch 2 times, most recently from 33bb9cf to ff73307 Compare August 6, 2019 22:44
@yf225 yf225 force-pushed the cpp_new_abi branch 2 times, most recently from d51156f to 1ab89ea Compare August 7, 2019 01:50
@pytorchbot pytorchbot added the module: build Build system issues label Aug 7, 2019
@yf225 yf225 force-pushed the cpp_new_abi branch 2 times, most recently from ac849a4 to bc44c4e Compare August 7, 2019 03:05
@yf225 yf225 force-pushed the cpp_new_abi branch 2 times, most recently from 71d255c to 4eb33e5 Compare August 11, 2019 22:59
This reverts commit 464de5f.
@yf225 yf225 force-pushed the cpp_new_abi branch 2 times, most recently from ec7c564 to de72d47 Compare August 17, 2019 03:57
@yf225 yf225 changed the title [WIP] Build libtorch binary with new ABI Build libtorch binary with new ABI Aug 17, 2019
@yf225 yf225 requested a review from ezyang August 17, 2019 04:43
return filtered_packaging_variants
else:
return packaging_variants
return [PackageFormatConfigNode(self, k, v) for k, v in self.py_tree.items()]
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice. I really hated the special casing here lol

@ezyang ezyang requested a review from kostmo August 17, 2019 18:59
@ezyang
Copy link
Contributor

ezyang commented Aug 17, 2019

@kostmo you should review the generator bits please

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.

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

@facebook-github-bot
Copy link
Contributor

@yf225 merged this pull request in 5aa0f89.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merged module: build Build system issues module: ci Related to continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants