Skip to content

Conversation

@peterjc123
Copy link
Collaborator

A possible fix for the problem stated in #12410.

@peterjc123
Copy link
Collaborator Author

Well, looks like there're bugs around the x64 compiler. Let's skip this and add x64 linker in PATH.

@peterjc123 peterjc123 changed the title Switch the default msvc toolchain to x64 version Add the x64 msvc toolchain into PATH Oct 8, 2018
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.

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

if "%REBUILD%"=="" ( pip install ninja )
call "C:\\Program Files (x86)\\Microsoft Visual Studio\\2017\\Community\\VC\\Auxiliary\\Build\\vcvarsall.bat" x64
call "C:\\Program Files (x86)\\Microsoft Visual Studio\\2017\\Community\\VC\\Auxiliary\\Build\\vcvarsall.bat" x86_amd64

This comment was marked as off-topic.

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants