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

[CMSIS-NN] Scalar to tensor constant pass to support only qnn.add and qnn.multiply #10563

Merged
merged 2 commits into from
Mar 14, 2022

Conversation

ashutosh-arm
Copy link
Contributor

Fixed scalar to tensor constant pass:

-Scalar to tensor pass to support qnn.add and qnn.multiply.
-Fixed application of the pass for non primary operands.
-Moved tests to check for int8 types specifically.

Co-authored-by: Luke Hutton <[email protected]>
Change-Id: If9cb41d0dd3f56666b6a2c0d9903502d3f9e4eae
@ashutosh-arm
Copy link
Contributor Author

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.

Few nits and a question

Change-Id: I67250a6214a2d54ef07d54d84eac4ce91474bb0e
@ashutosh-arm
Copy link
Contributor Author

Thanks @manupa-arm. PTAL - I have added another commit with review suggestions.

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! thanks @ashutosh-arm

@manupak manupak merged commit 5eb93df into apache:main Mar 14, 2022
pfk-beta pushed a commit to pfk-beta/tvm that referenced this pull request Apr 11, 2022
… qnn.multiply (apache#10563)

* Scalar to tensor constant pass to support qnn.add and qnn.multiply only.

Co-authored-by: Luke Hutton <[email protected]>
Change-Id: If9cb41d0dd3f56666b6a2c0d9903502d3f9e4eae

* Created a function to check if an expr is worthy of pass

Change-Id: I67250a6214a2d54ef07d54d84eac4ce91474bb0e

Co-authored-by: Luke Hutton <[email protected]>
@ashutosh-arm ashutosh-arm deleted the tuple_type_fix branch April 22, 2022 08:34
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