Skip to content

Conversation

@Lunderberg
Copy link
Contributor

This PR contains the non-relax portions of #13932 and #14132, which would be useful to have in the main branch of TVM. This upstreaming is part of the TVMScript round-trip improvements described here, specifically to support the I.module_attrs(...) annotations.

I've preserved the co-author listing for the two Unity PRs. Please let me know if there is anybody I missed, or if anybody would like to be removed.

@tvm-bot
Copy link
Collaborator

tvm-bot commented Apr 4, 2023

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

Generated by tvm-bot

@Lunderberg Lunderberg force-pushed the upstream_irmodule_parser_from_unity branch 2 times, most recently from 8226e16 to c1debda Compare April 4, 2023 17:54
Siyuan Feng and others added 2 commits April 5, 2023 10:35
This PR adds the TVMScript parser/ir_builder support based on the
blockbuilder.  This commit contains the non-relax portions from
apache#13932.

Co-authored-by: Ruihang Lai <[email protected]>
Co-authored-by: Junru Shao <[email protected]>
Co-authored-by: Tianqi Chen <[email protected]>
Co-authored-by: Yuchen Jin <[email protected]>
Co-authored-by: Steven S. Lyubomirsky <[email protected]>
Co-authored-by: Yong Wu <[email protected]>
This is an upstreaming of the non-relax portions of
apache#14132, including a unit test
specically to validate `I.module_attrs`.
@Lunderberg Lunderberg force-pushed the upstream_irmodule_parser_from_unity branch from a229184 to 94418bf Compare April 5, 2023 15:42
@Lunderberg Lunderberg force-pushed the upstream_irmodule_parser_from_unity branch from 94418bf to f845122 Compare April 5, 2023 15:42
@Lunderberg
Copy link
Contributor Author

@tvm-bot rerun

@tqchen tqchen merged commit b228037 into apache:main Apr 6, 2023
@Lunderberg Lunderberg deleted the upstream_irmodule_parser_from_unity branch April 7, 2023 14:02
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