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] Add pre-glow jit passes to remove exceptions and fuse Linear #3756

Closed
wants to merge 1 commit into from

Conversation

@jackm321
Copy link
Contributor

jackm321 commented Nov 8, 2019

Summary:
These two passes remove the only control flow in the BERT jit model

Documentation:
doxygen

Test Plan:
added unit tests
run on bert model and see that all control flow is gone

@jackm321 jackm321 force-pushed the jackm321:pt_run_bert branch from 9f7b12c to 2f0b0fd Nov 8, 2019
@jackm321 jackm321 requested review from yinghai, zrphercule and qizzzh and removed request for yinghai Nov 8, 2019
Copy link

facebook-github-bot left a comment

@jackm321 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@yinghai
yinghai approved these changes Nov 8, 2019
Copy link
Contributor

yinghai left a comment

LGTM

@jackm321 jackm321 force-pushed the jackm321:pt_run_bert branch from 2f0b0fd to 80d40fa Nov 8, 2019
Copy link

facebook-github-bot left a comment

@jackm321 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot

This comment has been minimized.

Copy link

facebook-github-bot commented Nov 8, 2019

@jackm321 merged this pull request in 50a5445.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.