[TFLite] Use structural checks in frontend tests#19888
Conversation
There was a problem hiding this comment.
Code Review
This pull request refactors the TFLite frontend tests by replacing fragile string-based IR assertions with robust structural equality checks against expected TVM Script IR modules. Feedback on the changes highlights two main issues: first, in test_reverse_sequence, the structural assertion was accidentally removed, leaving the expected module unused; second, in test_unique, the runtime execution test was completely replaced by the structural check, and it is recommended to retain both to preserve runtime test coverage.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
d913582 to
10f1025
Compare
This pr
verifyhelper unchanged and use directtvm.ir.assert_structural_equalwhere structural checks are needed.