Skip to content

[microNPU] modify the demo to use USMP#10511

Merged
leandron merged 2 commits intoapache:mainfrom
manupak:micronpu_demo_usmp
Mar 8, 2022
Merged

[microNPU] modify the demo to use USMP#10511
leandron merged 2 commits intoapache:mainfrom
manupak:micronpu_demo_usmp

Conversation

@manupak
Copy link
Contributor

@manupak manupak commented Mar 7, 2022

This commit changes the demo to use USMP.

This commit changes the demo to use USMP.

Change-Id: I87e85fdd9b2efea9559dda186c1644e4d63509ff
@manupak
Copy link
Contributor Author

manupak commented Mar 7, 2022

cc : @Mousius @grant-arm

@grant-arm
Copy link
Contributor

Thanks for this @manupa-arm ! This is fantastic.
Just one nit, otherwise it LGTM!

--executor=aot \
--executor-aot-interface-api=c \
--executor-aot-unpacked-api=1 \
--executor-aot-unpacked-api=1 \
Copy link
Contributor

Choose a reason for hiding this comment

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

This line is duplicated.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops. done!

* removing a repeated TVMC CLI argument

Change-Id: I3e1ec328155bfc4c9ce813224a7f06b09e66232c
@manupak
Copy link
Contributor Author

manupak commented Mar 7, 2022

Thanks for the quick feedback! @grant-arm . I did the change now.

@grant-arm
Copy link
Contributor

LGTM!

@manupak manupak requested review from Mousius and leandron March 8, 2022 10:13
Copy link
Contributor

@leandron leandron left a comment

Choose a reason for hiding this comment

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

LGTM

@leandron leandron merged commit acf8be7 into apache:main Mar 8, 2022
ziqiangxu8457 pushed a commit to ziqiangxu8457/tvm that referenced this pull request Mar 9, 2022
* [microNPU] modify the demo to use USMP

This commit changes the demo to use USMP.

Change-Id: I87e85fdd9b2efea9559dda186c1644e4d63509ff

* [microNPU] modify the demo to use USMP

* removing a repeated TVMC CLI argument

Change-Id: I3e1ec328155bfc4c9ce813224a7f06b09e66232c
pfk-beta pushed a commit to pfk-beta/tvm that referenced this pull request Apr 11, 2022
* [microNPU] modify the demo to use USMP

This commit changes the demo to use USMP.

Change-Id: I87e85fdd9b2efea9559dda186c1644e4d63509ff

* [microNPU] modify the demo to use USMP

* removing a repeated TVMC CLI argument

Change-Id: I3e1ec328155bfc4c9ce813224a7f06b09e66232c
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