Skip to content

[TIRx] Fix stale Simplify import in lowering test#19642

Merged
guan404ming merged 1 commit into
apache:mainfrom
tlopex:fix-tirx-stmtsimplify
May 30, 2026
Merged

[TIRx] Fix stale Simplify import in lowering test#19642
guan404ming merged 1 commit into
apache:mainfrom
tlopex:fix-tirx-stmtsimplify

Conversation

@tlopex

@tlopex tlopex commented May 30, 2026

Copy link
Copy Markdown
Member

test_transform_lower_tirx.py imports and calls Simplify, but the pass is named StmtSimplify (the only simplify pass exported from tvm.tirx.transform). The stale name makes the module fail to import at collection time. Use StmtSimplify so the test collects and runs.

test_transform_lower_tirx.py imports and calls Simplify, but the pass is
named StmtSimplify (the only simplify pass exported from
tvm.tirx.transform). The stale name makes the module fail to import at
collection time. Use StmtSimplify so the test collects and runs.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the test file test_transform_lower_tirx.py to import and use StmtSimplify instead of Simplify from tvm.tirx.transform. There are no review comments, so I have no feedback to provide.

@guan404ming guan404ming merged commit 7a9f568 into apache:main May 30, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants