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

[Not for Landing] Update FBGEMM to the latest version with VNNI instruction support and latest ASMJIT version #24051

Closed
wants to merge 1 commit into from

Conversation

jianyuh
Copy link
Member

@jianyuh jianyuh commented Aug 8, 2019

We would like to update FBGEMM with VNNI instruction support and newer version of ASMJIT support:
pytorch/FBGEMM#114

The CMake version incompatibility issue has been resolved in
asmjit/asmjit#252

This PR is purely for checking if our FBGEMM PR (pytorch/FBGEMM#114) will break PyTorch OSS test. It is not intended for landing.

@jianyuh jianyuh changed the title Update FBGEMM to the latest version with VNNI instruction support and… [Not for Landing] Update FBGEMM to the latest version with VNNI instruction support and… Aug 9, 2019
@jianyuh jianyuh changed the title [Not for Landing] Update FBGEMM to the latest version with VNNI instruction support and… [Not for Landing] Update FBGEMM to the latest version with VNNI instruction support and latest ASMJIT version Aug 9, 2019
@jianyuh jianyuh requested review from soumith, ezyang and gchanan August 9, 2019 03:40
@jianyuh
Copy link
Member Author

jianyuh commented Aug 9, 2019

The newer version of FBGEMM with the latest ASMJIT (After the CMake fix: asmjit/asmjit#252) is not going to break OSS build. We are going to ship this PR (pytorch/FBGEMM#114) in FBGEMM.

@jianyuh
Copy link
Member Author

jianyuh commented Aug 9, 2019

The reason for the conflict: we have a recent update in FBGEMM (pytorch/FBGEMM#112), which triggered a commit ID update in PyTorch. This conflict will be resolved by rebase.

@jianyuh
Copy link
Member Author

jianyuh commented Aug 9, 2019

Close this PR as we have landed the FBGEMM PR (pytorch/FBGEMM#114) and this FBGEMM commit ID has been updated in a6c3a95.

@jianyuh jianyuh closed this Aug 9, 2019
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.

2 participants