Skip to content

[BYOC][ACL] Support add operation#6532

Merged
comaniac merged 3 commits intoapache:masterfrom
d-smirnov:acl-add-operation
Oct 11, 2020
Merged

[BYOC][ACL] Support add operation#6532
comaniac merged 3 commits intoapache:masterfrom
d-smirnov:acl-add-operation

Conversation

@d-smirnov
Copy link
Copy Markdown
Contributor

Added support for an "add" operation implemented via ACL + unit test

@comaniac comaniac self-assigned this Sep 22, 2020
Comment thread python/tvm/relay/op/contrib/arm_compute_lib.py Outdated
Comment thread python/tvm/relay/op/contrib/arm_compute_lib.py Outdated
Comment thread tests/python/contrib/test_arm_compute_lib/test_add.py
Comment thread tests/python/contrib/test_arm_compute_lib/test_add.py Outdated
Comment thread tests/python/contrib/test_arm_compute_lib/test_add.py Outdated
@comaniac comaniac changed the title ACL support: "add" operation [BYOC][ACL] Support add operation Sep 22, 2020
@comaniac comaniac added the status: need update need update based on feedbacks label Sep 22, 2020
Comment thread src/runtime/contrib/arm_compute_lib/acl_runtime.cc
Comment thread python/tvm/relay/op/contrib/arm_compute_lib.py Outdated
Comment thread python/tvm/relay/op/contrib/arm_compute_lib.py Outdated
Comment thread tests/python/contrib/test_arm_compute_lib/test_add.py Outdated
Comment thread tests/python/contrib/test_arm_compute_lib/test_add.py Outdated
Comment thread tests/python/contrib/test_arm_compute_lib/test_add.py
Comment thread tests/python/contrib/test_arm_compute_lib/test_add.py Outdated
@d-smirnov d-smirnov force-pushed the acl-add-operation branch 4 times, most recently from cbfcbe0 to 2cc9ebe Compare September 28, 2020 16:31
Comment thread python/tvm/relay/op/contrib/arm_compute_lib.py Outdated
Comment thread python/tvm/relay/op/contrib/arm_compute_lib.py Outdated
Comment thread src/runtime/contrib/arm_compute_lib/acl_runtime.cc Outdated
Comment thread src/runtime/contrib/arm_compute_lib/acl_runtime.cc Outdated
Copy link
Copy Markdown
Contributor

@comaniac comaniac left a comment

Choose a reason for hiding this comment

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

LGTM. Please fix the CI errors.

Added support for an "add" operation implemented via ACL
for fp32 and quantized uint8 data types
Copy link
Copy Markdown
Contributor

@lhutton1 lhutton1 left a comment

Choose a reason for hiding this comment

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

Almost there :) just a couple of small things and a concern about a saturated test.

Could we also add support for add to the documentation here: https://github.com/apache/incubator-tvm/blob/master/docs/deploy/arm_compute_lib.rst

Comment thread tests/python/contrib/test_arm_compute_lib/test_add.py Outdated
Comment thread tests/python/contrib/test_arm_compute_lib/test_add.py Outdated
Comment thread src/runtime/contrib/arm_compute_lib/acl_runtime.cc Outdated
@comaniac
Copy link
Copy Markdown
Contributor

comaniac commented Oct 7, 2020

@d-smirnov would you address the comments from @lhutton1 or?

Copy link
Copy Markdown
Contributor

@lhutton1 lhutton1 left a comment

Choose a reason for hiding this comment

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

LGTM

@comaniac comaniac merged commit e561007 into apache:master Oct 11, 2020
@comaniac
Copy link
Copy Markdown
Contributor

Thanks @d-smirnov @lhutton1

@comaniac comaniac added status: accepted and removed status: need update need update based on feedbacks labels Oct 11, 2020
TusharKanekiDey pushed a commit to TusharKanekiDey/tvm that referenced this pull request Oct 13, 2020
* [BYOC][ACL] Support add operation

Added support for an "add" operation implemented via ACL
for fp32 and quantized uint8 data types

* Addressed lhutton1 comments

* linter
TusharKanekiDey pushed a commit to TusharKanekiDey/tvm that referenced this pull request Oct 14, 2020
* [BYOC][ACL] Support add operation

Added support for an "add" operation implemented via ACL
for fp32 and quantized uint8 data types

* Addressed lhutton1 comments

* linter
TusharKanekiDey pushed a commit to TusharKanekiDey/tvm that referenced this pull request Oct 15, 2020
* [BYOC][ACL] Support add operation

Added support for an "add" operation implemented via ACL
for fp32 and quantized uint8 data types

* Addressed lhutton1 comments

* linter
TusharKanekiDey pushed a commit to TusharKanekiDey/tvm that referenced this pull request Oct 15, 2020
* [BYOC][ACL] Support add operation

Added support for an "add" operation implemented via ACL
for fp32 and quantized uint8 data types

* Addressed lhutton1 comments

* linter
TusharKanekiDey pushed a commit to TusharKanekiDey/tvm that referenced this pull request Oct 16, 2020
* [BYOC][ACL] Support add operation

Added support for an "add" operation implemented via ACL
for fp32 and quantized uint8 data types

* Addressed lhutton1 comments

* linter
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request Oct 19, 2020
* [BYOC][ACL] Support add operation

Added support for an "add" operation implemented via ACL
for fp32 and quantized uint8 data types

* Addressed lhutton1 comments

* linter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants