Solve compiler errors in decode op - #3284
Merged
Merged
Conversation
1. When building with old Xtensa toolchains, the compiler throw an unused variable warning which is treated as error (-Werror is defined by default). The cause is ScopedMicroProfile instantiation in the decoder op. Added a dummy reference to mute the warning. 2. Old pre C++14 Xtensa compilers don't support ticks in preprocessor constants, e.g. 0x8000'0000 Removed the ticks. BUG=451462435
veblush
approved these changes
Jan 9, 2026
shlmregev
added a commit
to shlmregev/tflite-micro
that referenced
this pull request
Jan 12, 2026
1. When building with old Xtensa toolchains, the compiler throw an unused variable warning which is treated as error (-Werror is defined by default). The cause is ScopedMicroProfile instantiation in the decoder op. Added a dummy reference to mute the warning. 2. Old pre C++14 Xtensa compilers don't support ticks in preprocessor constants, e.g. 0x8000'0000 Removed the ticks. BUG=451462435
github-merge-queue Bot
pushed a commit
that referenced
this pull request
Jan 14, 2026
* Add missing optimizations for xtensa/ and hexagon/ to forntend ops under signal/ * Revert change to copyright year * Fix wrong include paths * Reorder include paths * Move static inline functions out of the namespace block. * Fix path to KissFFT * Remove file that was added by mistake * Realized a lot of these optimizations were already added under src/ * Reverted a few more files * Fix compilation errors * Porting Reduce_All reference operator porting from TFLite to TFLM (#3269) * Sync files related to Reverse_V2 from TFLite #3110 * PRelu Int16x8 support in RefC * Fix code style in prelu_test.cc * 1. Reverted the copyright year * Resolved compilation error for Int8x8 test case * Add Dynamic_Update_Slice support to TFLM * Code style error correction * Code style correction * Replaced hard coded MaxDimensions to RuntimeShape::kMaxSmallSize * 1. Added more test cases \n2.Removed unused code * Updates for test failure on ARM * Code style updates * Updates on test case failure for ARM * Updates on test case failure for ARM * Code style updates * Add Reduce_All reference operator support to TFLM * Resolving HiFi build errors --------- Co-authored-by: Esun Kim <veblush@google.com> * Provide default values for uninitialized variable (#3282) Switch-cases in decode_state_lut.cc don't assign a default value to a local variables. On one version of ARM GCC (building for ARM cortex m33, this results in a compiler error [-Werror=maybe-uninitialized]. BUG=451462435 * Add Ingenic MIPS port to README.md (#3255) * Add Ingenic MIPS port to README.md * Update README.md Changed order of new board to be alphabetical * Update README.md Removed extra brackets making things look wrong. --------- Co-authored-by: Esun Kim <veblush@google.com> * Solve compiler errors in decode op (#3284) 1. When building with old Xtensa toolchains, the compiler throw an unused variable warning which is treated as error (-Werror is defined by default). The cause is ScopedMicroProfile instantiation in the decoder op. Added a dummy reference to mute the warning. 2. Old pre C++14 Xtensa compilers don't support ticks in preprocessor constants, e.g. 0x8000'0000 Removed the ticks. BUG=451462435 * Fix coding style * Match FFT int16 test's tolerance with audiofrontend's * Fix failure in micro_speech example * Increase tolerance of feature extraction * Incerased error interval for micro speech detection * Incerased error interval for micro speech detection * Incerased error interval for micro speech detection * Incerased error interval for micro speech detection * Comment on looser tolerance in micro speech test and limit it to Xtensa --------- Co-authored-by: Kunasi Ramesh <28750242+rameshkunasi@users.noreply.github.com> Co-authored-by: Esun Kim <veblush@google.com> Co-authored-by: Matthew Vance <yinzara@gmail.com>
unmeshna017
pushed a commit
to unmeshna017/tflite-micro
that referenced
this pull request
Feb 10, 2026
1. When building with old Xtensa toolchains, the compiler throw an unused variable warning which is treated as error (-Werror is defined by default). The cause is ScopedMicroProfile instantiation in the decoder op. Added a dummy reference to mute the warning. 2. Old pre C++14 Xtensa compilers don't support ticks in preprocessor constants, e.g. 0x8000'0000 Removed the ticks. BUG=451462435
unmeshna017
pushed a commit
to unmeshna017/tflite-micro
that referenced
this pull request
Feb 10, 2026
* Add missing optimizations for xtensa/ and hexagon/ to forntend ops under signal/ * Revert change to copyright year * Fix wrong include paths * Reorder include paths * Move static inline functions out of the namespace block. * Fix path to KissFFT * Remove file that was added by mistake * Realized a lot of these optimizations were already added under src/ * Reverted a few more files * Fix compilation errors * Porting Reduce_All reference operator porting from TFLite to TFLM (tensorflow#3269) * Sync files related to Reverse_V2 from TFLite tensorflow#3110 * PRelu Int16x8 support in RefC * Fix code style in prelu_test.cc * 1. Reverted the copyright year * Resolved compilation error for Int8x8 test case * Add Dynamic_Update_Slice support to TFLM * Code style error correction * Code style correction * Replaced hard coded MaxDimensions to RuntimeShape::kMaxSmallSize * 1. Added more test cases \n2.Removed unused code * Updates for test failure on ARM * Code style updates * Updates on test case failure for ARM * Updates on test case failure for ARM * Code style updates * Add Reduce_All reference operator support to TFLM * Resolving HiFi build errors --------- Co-authored-by: Esun Kim <veblush@google.com> * Provide default values for uninitialized variable (tensorflow#3282) Switch-cases in decode_state_lut.cc don't assign a default value to a local variables. On one version of ARM GCC (building for ARM cortex m33, this results in a compiler error [-Werror=maybe-uninitialized]. BUG=451462435 * Add Ingenic MIPS port to README.md (tensorflow#3255) * Add Ingenic MIPS port to README.md * Update README.md Changed order of new board to be alphabetical * Update README.md Removed extra brackets making things look wrong. --------- Co-authored-by: Esun Kim <veblush@google.com> * Solve compiler errors in decode op (tensorflow#3284) 1. When building with old Xtensa toolchains, the compiler throw an unused variable warning which is treated as error (-Werror is defined by default). The cause is ScopedMicroProfile instantiation in the decoder op. Added a dummy reference to mute the warning. 2. Old pre C++14 Xtensa compilers don't support ticks in preprocessor constants, e.g. 0x8000'0000 Removed the ticks. BUG=451462435 * Fix coding style * Match FFT int16 test's tolerance with audiofrontend's * Fix failure in micro_speech example * Increase tolerance of feature extraction * Incerased error interval for micro speech detection * Incerased error interval for micro speech detection * Incerased error interval for micro speech detection * Incerased error interval for micro speech detection * Comment on looser tolerance in micro speech test and limit it to Xtensa --------- Co-authored-by: Kunasi Ramesh <28750242+rameshkunasi@users.noreply.github.com> Co-authored-by: Esun Kim <veblush@google.com> Co-authored-by: Matthew Vance <yinzara@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
When building with old Xtensa toolchains, the compiler throw an unused variable warning which is treated as error (-Werror is defined by default). The cause is ScopedMicroProfile instantiation in the decoder op. Added a dummy reference to mute the warning.
Old pre C++14 Xtensa compilers don't support ticks in preprocessor constants, e.g. 0x8000'0000 Removed the ticks.
BUG=451462435