-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
DNNL-BYOC enhancement #9797
DNNL-BYOC enhancement #9797
Conversation
e1a4a1b
to
b2abb2b
Compare
449b68a
to
ed87f71
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this change, it looks really excellent. I have a few small comments but am curious if youve done any benchmarking with this new pattern fusion. If so, how much do these changes speed up models you tried?
Thank you for your comments. We do have done some benchmark, and the related results are posted on RFC. Note that, the results in |
ed87f71
to
a28ce62
Compare
* add unit test for byoc-dnnl * add byoc-dnnl pattern and their test cases
* add unit test for byoc-dnnl * add byoc-dnnl pattern and their test cases
dnnl.py
.test_dnnl.py
.dnnl_json_runtime.cc
.dilated conv2d
with dnnl backend.