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

[usmp] U3 use case #11015

Merged
merged 3 commits into from
Jun 28, 2022
Merged

[usmp] U3 use case #11015

merged 3 commits into from
Jun 28, 2022

Conversation

d-smirnov
Copy link
Contributor

@d-smirnov d-smirnov commented Apr 14, 2022

This PR adds tir.constant U3 use case functionality allowing user to supply own set of constants to constant pools. TVM provides macros with length and data for user to supply this to the runner.

@d-smirnov d-smirnov force-pushed the U3-usecase branch 4 times, most recently from cc86ef9 to f0a6b67 Compare June 20, 2022 07:56
@d-smirnov d-smirnov force-pushed the U3-usecase branch 8 times, most recently from cc24d4b to f538729 Compare June 23, 2022 13:15
@d-smirnov d-smirnov marked this pull request as ready for review June 23, 2022 16:56
@d-smirnov d-smirnov changed the title U3 usecase [usmp] U3 use case Jun 23, 2022
Copy link
Contributor

@manupak manupak left a comment

Choose a reason for hiding this comment

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

Thanks @d-smirnov . The implementation seems good!

I had few concerns and hope they would be minor ones to address.

src/target/source/interface_c.cc Outdated Show resolved Hide resolved
src/target/source/codegen_params.cc Show resolved Hide resolved
src/target/source/interface_c.cc Show resolved Hide resolved
src/target/source/interface_c.cc Show resolved Hide resolved
tests/cpp/target/source/interface_c_test.cc Outdated Show resolved Hide resolved
tests/python/relay/aot/test_crt_aot_usmp.py Outdated Show resolved Hide resolved
d-smirnov and others added 3 commits June 27, 2022 23:28
Change-Id: Ibc088f19ad1dc9466fc368f8523baa30ee88b7d0
Added unit test for InterfaceCNode::EmitConstantPool method
Copy link
Contributor

@manupak manupak left a comment

Choose a reason for hiding this comment

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

LGTM!

@manupak manupak merged commit 08723d0 into apache:main Jun 28, 2022
@manupak
Copy link
Contributor

manupak commented Jun 28, 2022

Thanks @d-smirnov -- This is merged now!

blackkker pushed a commit to blackkker/tvm that referenced this pull request Jul 7, 2022
* U3

Change-Id: Ibc088f19ad1dc9466fc368f8523baa30ee88b7d0

* addressed upstream comments

* Unit test added

Added unit test for InterfaceCNode::EmitConstantPool method
masahi pushed a commit to masahi/tvm that referenced this pull request Jul 15, 2022
* U3

Change-Id: Ibc088f19ad1dc9466fc368f8523baa30ee88b7d0

* addressed upstream comments

* Unit test added

Added unit test for InterfaceCNode::EmitConstantPool method
mikeseven pushed a commit to mikeseven/tvm that referenced this pull request Sep 27, 2023
* U3

Change-Id: Ibc088f19ad1dc9466fc368f8523baa30ee88b7d0

* addressed upstream comments

* Unit test added

Added unit test for InterfaceCNode::EmitConstantPool method
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