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
add support for quantized ops on windows #14955
Conversation
Can one of the admins verify this patch? |
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.
Woohoo! Thanks for doing this, Guenther!
@tensorflow-jenkins test this please. |
yeah, this was in the making for some time. thanks for helping with the gemmlowp. |
Yes, I think those failures are unrelated. Thanks again! @tensorflow-jenkins test this please. |
Jenkins, test this please. |
@tensorflow-jenkins test this please |
Quantized ops support on windows. Added some simple python unit test to make sure it works.
Gotcha: I need to use the latest version of gemmlowp in gemmlowp.cmake which is not on mirror.bazel.build so I use github.com directly for now.
This should provide the same functionality as tf on linux. Performance could be better because msvc can't make use of the inline asm in gemmlowp (compiling gemmlowp with clang would show much better results).