-
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
[TOPI][Hexagon] Implement quantized elementwise for hexagon #12606
[TOPI][Hexagon] Implement quantized elementwise for hexagon #12606
Conversation
b09ef6e
to
f24d303
Compare
f24d303
to
0eb2d8b
Compare
0eb2d8b
to
293888c
Compare
Hi, @mehrdadh @kparzysz-quic @TejashShah this PR is ready for review. Thank you |
@trahman-quic @jverma-quic thanks for the contribution and review! |
c8d3750
to
4d7bc6e
Compare
e2a7d85
to
6acc3b8
Compare
9527f92
to
56eb957
Compare
1d89d43
to
5d06698
Compare
5d06698
to
f22ecdd
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 the PR, Tasmia! It looks good to me.
Thank you @jverma-quic . |
…2606) * [TOPI][Hexagon] Add test and schedule for uint8 resize2d * Fix correctness issue * Reformat * [TOPI][Hexagon] Implement quantized elementwise * Reformat * Address review comments * Reformat * Revert * Address review comments
This PR adds implementation for quantized elementwise ops (add, subtract, multiply) for hexagon with support of broadcasting. The quantization method uses fixed-point number to replace floating point instructions. In addition to the implementation, it includes appropriate schedules for uint8 layout. It also adds test case for these ops in uint8 dtype.
cc @mehrdadh