Skip to content

[ETHOSN] Remove backwards copy in the runtime#12968

Merged
leandron merged 1 commit intoapache:mainfrom
lhutton1:remove-back-copy
Oct 6, 2022
Merged

[ETHOSN] Remove backwards copy in the runtime#12968
leandron merged 1 commit intoapache:mainfrom
lhutton1:remove-back-copy

Conversation

@lhutton1
Copy link
Contributor

@lhutton1 lhutton1 commented Oct 3, 2022

The runtime had a backwards copy which was important historically to work around a bug in the NPU driver stack. It should now be safe to replace with a standard copy operation.

Some cleanup has also been completed to remove unused functionality from the runtime.

cc @Leo-arm @ashutosh-arm @leandron

@github-actions github-actions bot requested a review from leandron October 3, 2022 09:50
The runtime had a backwards copy which was important historically to
work around a bug in the NPU driver stack. It should now be safe to
replace with a standard copy operation.

Some cleanup has also been completed to remove unused functionality
from the runtime.

Change-Id: I8b800b6a57dca40ea5b9538a8874489e530caa61
Copy link
Contributor

@leandron leandron 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 @lhutton1 @Leo-arm!

@leandron leandron merged commit 6147c31 into apache:main Oct 6, 2022
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
The runtime had a backwards copy which was important historically to
work around a bug in the NPU driver stack. It should now be safe to
replace with a standard copy operation.

Some cleanup has also been completed to remove unused functionality
from the runtime.

Change-Id: I8b800b6a57dca40ea5b9538a8874489e530caa61
@lhutton1 lhutton1 deleted the remove-back-copy branch February 13, 2023 19:35
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