Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TVMSCRIPT] Misc error message improvements #9543

Merged

Conversation

tkonolige
Copy link
Contributor

This PR contains a couple of better error messages I've added to the tvmscript parser.

Copy link
Member

@junrushao junrushao left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@Hzfengsy Hzfengsy left a comment

Choose a reason for hiding this comment

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

LGTM. Let's merge it when CI is green

@tkonolige tkonolige force-pushed the tvmscript_errormessage_improvements branch from 6f1a2c6 to 96a907d Compare November 23, 2021 17:15
@tkonolige tkonolige force-pushed the tvmscript_errormessage_improvements branch from 96a907d to e75f13b Compare December 2, 2021 23:27
@tkonolige
Copy link
Contributor Author

@Hzfengsy @junrushao1994 CI is green. Can we merge?

@junrushao
Copy link
Member

Of course! Thanks for improving the error message!

@junrushao junrushao merged commit 7f683da into apache:main Dec 3, 2021
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 7, 2022
* [TVMSCRIPT] Misc error message improvements

* only prevent indexing into handles with multiple indexes

* lint
yangulei pushed a commit to yangulei/tvm that referenced this pull request Jan 11, 2022
* [TVMSCRIPT] Misc error message improvements

* only prevent indexing into handles with multiple indexes

* lint
yangulei pushed a commit to yangulei/tvm that referenced this pull request Jan 12, 2022
* [TVMSCRIPT] Misc error message improvements

* only prevent indexing into handles with multiple indexes

* lint
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 13, 2022
* [TVMSCRIPT] Misc error message improvements

* only prevent indexing into handles with multiple indexes

* lint
qsqqsqqsq-intellif pushed a commit to qsqqsqqsq-intellif/tvm that referenced this pull request Apr 29, 2022
* [TVMSCRIPT] Misc error message improvements

* only prevent indexing into handles with multiple indexes

* lint
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.

3 participants