Skip to content

[Unity] Improved error message in tvm::relax::UpdateStructInfo - #16472

Merged
slyubomirsky merged 1 commit into
apache:mainfrom
Lunderberg:relax_improved_update_structinfo_error_message
Jan 26, 2024
Merged

slyubomirsky merged 1 commit into
apache:mainfrom
Lunderberg:relax_improved_update_structinfo_error_message

Conversation

@Lunderberg

Copy link
Copy Markdown
Contributor

If the struct info cannot be updated, provide the expression, the previous struct info, and the new struct info in the error message.

If the struct info cannot be updated, provide the expression, the
previous struct info, and the new struct info in the error message.

@slyubomirsky slyubomirsky 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.

Thanks! It's a handy improvement.

@slyubomirsky
slyubomirsky merged commit 8ade9c3 into apache:main Jan 26, 2024
@Lunderberg

Copy link
Copy Markdown
Contributor Author

Thank you! Any time that I need more information for debugging, I try to add it to the error message so that it’s easier the next time around.

@Lunderberg
Lunderberg deleted the relax_improved_update_structinfo_error_message branch January 26, 2024 14:50
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