Skip to content

[Relax][Transform] Add SelectNode handling in SymbolicMatcher#17368

Merged
MasterJH5574 merged 1 commit into
apache:mainfrom
mengshyu:0912-selectnode
Sep 15, 2024
Merged

[Relax][Transform] Add SelectNode handling in SymbolicMatcher#17368
MasterJH5574 merged 1 commit into
apache:mainfrom
mengshyu:0912-selectnode

Conversation

@mengshyu

Copy link
Copy Markdown
Contributor

This PR added support for handling SelectNode in the SymbolicMatcher class by modifying the VisitExpr_ function to match the true_value and false_value expressions between the current SelectNode and the other expression. If the other expression is not a SelectNode, the matching condition is updated to ensure the current SelectNode expression is equivalent to the other expression.

@tqchen

tqchen commented Sep 12, 2024

Copy link
Copy Markdown
Member

@mengshyu please fix lint

This PR added support for handling SelectNode in the SymbolicMatcher
class by modifying the VisitExpr_ function to match the true_value
and false_value expressions between the current SelectNode and the
other expression. If the other expression is not a SelectNode, the
matching condition is updated to ensure the current SelectNode
expression is equivalent to the other expression.
@MasterJH5574

Copy link
Copy Markdown
Contributor

@tvm-bot rerun

@tqchen

tqchen commented Sep 13, 2024

Copy link
Copy Markdown
Member

CI issue should be unblocked by #17371

@CharlieFRuan

Copy link
Copy Markdown
Member

@tvm-bot rerun

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.

4 participants