-
Notifications
You must be signed in to change notification settings - Fork 954
Support for DECODE operator #3190
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
Conversation
@tensorflow/micro Add initial support for DECODE operator. Add reference implementation. Add LUT decompression support. Update op resolvers. Update Makefiles and Bazel BUILD files. Add kernel unit test. bug=fixes tensorflow#3131
…em). Cleanup unit test.
@tensorflow/micro Additional support for DECODE operator. Add Xtensa optimizations for LUT decompression. Move all Xtensa kernel source references to the Xtensa target makefile. bug=fixes tensorflow#3150
@tensorflow/micro Reorganize Xtensa makefiles such that all references to optimized kernel sources are moved to the Xtensa target makefile. Move hifimini kernel sources to the parent directory, and rename them so they do not interfere with the target overlay mechanism of the root makefile. bug=fixes tensorflow#3153
Fix code style errors.
@tensorflow/micro Add reference implementation of pruning to DECODE operator. Makefile and Bazel BUILD file changes. Additional unit tests. bug=fixes tensorflow#3161
@tensorflow/micro Add optimized xtensa implementation of pruning to DECODE operator. Makefile changes. Additional unit tests. bug=fixes tensorflow#3171
@tensorflow/micro Add reference implementation of Huffman decompression to DECODE operator. Makefile and Bazel BUILD file changes. Additional unit tests. bug=fixes tensorflow#3187
@tensorflow/micro Add optimized xtensa implementation of Huffman decompression to DECODE operator. Makefile changes. bug=fixes tensorflow#3189
Remove xtensa optimizations.
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.
I've reviewed the non-optimization parts of this PR (very small) and LGTM.
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.
Approving. Relying on CI and the unit tests for the XTensa-specific implementation.
@tensorflow/micro
Add optimized xtensa implementation of these decompression types to DECODE operator:
Makefile changes.
bug=fixes #3189