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

[AoT]Add get_input_name function to AoT Module #14071

Merged
merged 4 commits into from
Feb 22, 2023

Conversation

mehrdadh
Copy link
Member

@mehrdadh mehrdadh commented Feb 21, 2023

This PR adds get_input_name function to AoT. This functionality would help with features like TVMC where we need to set the input without asking the user to specify the input name.

This PR also adds create_local_aot_executor for micro and refactors that across the codebase for all CRT use cases.
In addition this PR fixes a bug in python/tvm/relay/build_module.py where relay.build for AotExecutor was using graph executor instead of aot.

@tvm-bot
Copy link
Collaborator

tvm-bot commented Feb 21, 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

@mehrdadh mehrdadh requested a review from areusch February 21, 2023 21:46
@mehrdadh
Copy link
Member Author

cc @alanmacd for review

@mehrdadh mehrdadh force-pushed the aot/add_get_input_name branch from efb6707 to 89ace74 Compare February 22, 2023 00:34
@masahi masahi merged commit b732e3c into apache:main Feb 22, 2023
@mehrdadh mehrdadh deleted the aot/add_get_input_name branch February 22, 2023 21:40
yongwww pushed a commit to yongwww/tvm that referenced this pull request Feb 27, 2023
* Add get_input_name to C AOT

* add get_input_name to AOT C++

* lint

* fix bug in AotExecutor
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