Skip to content

Conversation

@lhutton1
Copy link
Contributor

@lhutton1 lhutton1 commented Feb 1, 2022

Adds support for offloading an int8 Leaky ReLU activation function to the NPU by leagalizing to a LUT.

@lhutton1
Copy link
Contributor Author

lhutton1 commented Feb 1, 2022

@lhutton1 lhutton1 force-pushed the leaky-relu branch 2 times, most recently from 6537626 to 821b232 Compare February 4, 2022 11:26
Copy link
Contributor

@NicolaLancellotti NicolaLancellotti left a comment

Choose a reason for hiding this comment

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

LGTM!

@lhutton1 lhutton1 force-pushed the leaky-relu branch 2 times, most recently from 4e1f129 to 78febc3 Compare February 12, 2022 14:41
Adds support for offloading an int8 Leaky ReLU activation function
to the NPU by legalizing to a LUT.

Change-Id: I63dd5b16a1a2a747b11f15a5b8124810e2ebf491
Change-Id: I35b59200b16a7eff1915f771ab6b5d9181d4f3ab
@lhutton1
Copy link
Contributor Author

Friendly ping for comments/approval

Copy link
Contributor

@ekalda ekalda left a comment

Choose a reason for hiding this comment

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

Thanks @lhutton1, looks all good to me!

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
Copy link
Contributor

manupak commented Feb 17, 2022

Thanks all!

@manupak manupak merged commit 93cbadb into apache:main Feb 17, 2022
@lhutton1 lhutton1 deleted the leaky-relu branch March 9, 2022 10:17
pfk-beta pushed a commit to pfk-beta/tvm that referenced this pull request Apr 11, 2022
* [microNPU] Add support for LeakyReLU

Adds support for offloading an int8 Leaky ReLU activation function
to the NPU by legalizing to a LUT.

Change-Id: I63dd5b16a1a2a747b11f15a5b8124810e2ebf491

* refactor LeakyReLUParams to inherit from LutActivationParams

Change-Id: I35b59200b16a7eff1915f771ab6b5d9181d4f3ab
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.

4 participants