Skip to content

Conversation

@Mousius
Copy link
Member

@Mousius Mousius commented Sep 30, 2021

This patch aims to add initial support for the Add operator to CMSIS NN, which was actually similar enough to the Mul operator that it shares quite a bit of code - exciting times.

@asparkhi
Copy link
Contributor

asparkhi commented Oct 1, 2021

LGTM 💯

This patch aims to add initial support for the `Add` operator to CMSIS NN, which was actually similar enough to the `Mul` operator that it shares quite a bit of code - exciting times.
Copy link
Contributor

@areusch areusch left a comment

Choose a reason for hiding this comment

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

thanks @Mousius @ashutosh-arm !

@areusch areusch merged commit 6f98e2d into apache:main Oct 4, 2021
@Mousius Mousius deleted the cmsis-add-operator branch October 5, 2021 09:36
Mousius added a commit to Mousius/tvm that referenced this pull request Oct 5, 2021
apache#9179 and apache#9167 both landed at the same time which resulted in new tests without the skipping.
masahi pushed a commit that referenced this pull request Oct 6, 2021
#9179 and #9167 both landed at the same time which resulted in new tests without the skipping.
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 7, 2022
This patch aims to add initial support for the `Add` operator to CMSIS NN, which was actually similar enough to the `Mul` operator that it shares quite a bit of code - exciting times.
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 7, 2022
…e#9200)

apache#9179 and apache#9167 both landed at the same time which resulted in new tests without the skipping.
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 13, 2022
This patch aims to add initial support for the `Add` operator to CMSIS NN, which was actually similar enough to the `Mul` operator that it shares quite a bit of code - exciting times.
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 13, 2022
…e#9200)

apache#9179 and apache#9167 both landed at the same time which resulted in new tests without the skipping.
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