Skip to content

Env variable to not check compiler abi#12708

Closed
goldsborough wants to merge 2 commits intopytorch:masterfrom
goldsborough:dont-check-abi
Closed

Env variable to not check compiler abi#12708
goldsborough wants to merge 2 commits intopytorch:masterfrom
goldsborough:dont-check-abi

Conversation

@goldsborough
Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Member

@fmassa fmassa left a comment

Choose a reason for hiding this comment

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

This looks good to me.
One thing that might be useful doing would be to have a slightly more robust env checking, like the one we use in setup.py, but this might be an overkill.

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.

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

@goldsborough goldsborough deleted the dont-check-abi branch October 22, 2018 12:31
@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:
For pytorch#10114

soumith fmassa
Pull Request resolved: pytorch#12708

Differential Revision: D10444102

Pulled By: goldsborough

fbshipit-source-id: 529e737e795bd8801beab2247be3dad296af5a3e
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