Skip to content

[BugFix][TVMScript] Add doc.keyword handling for ExprEvaluator._visit#19352

Merged
tlopex merged 2 commits into
apache:mainfrom
liggest:fix_eval_keywords
Apr 5, 2026
Merged

[BugFix][TVMScript] Add doc.keyword handling for ExprEvaluator._visit#19352
tlopex merged 2 commits into
apache:mainfrom
liggest:fix_eval_keywords

Conversation

@liggest

@liggest liggest commented Apr 5, 2026

Copy link
Copy Markdown
Contributor

Add handling for doc.keyword nodes in ExprEvaluator._visit to ensure expressions (e.g. BoolOp) in keyword arguments are processed with correct evaluation methods.

Fix #18972 . For more details, please refer to this issue.

@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 TVM script parser's evaluator to handle doc.keyword nodes by recursively visiting their values. I have no feedback to provide as there are no review comments.

@tlopex tlopex left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! Thank you for the contribution!

@tlopex tlopex merged commit 36a82f5 into apache:main Apr 5, 2026
6 checks passed
Aharrypotter pushed a commit to Aharrypotter/tvm that referenced this pull request Apr 10, 2026
…sit` (apache#19352)

Add handling for `doc.keyword` nodes in `ExprEvaluator._visit` to ensure
expressions (e.g. `BoolOp`) in keyword arguments are processed with
correct evaluation methods.

Fix apache#18972 . For more details, please refer to this issue.
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.

[Bug] Tirx's prim_func handles the and operator inconsistently in positional vs keyword arguments of a function call

2 participants