Skip to content

Fix compile time and runtime errors of EdgeTPURuntime - #8133

Merged
masahi merged 3 commits into
apache:mainfrom
akmaru:fix-error-edgetpu-runtime
Jun 11, 2021
Merged

Fix compile time and runtime errors of EdgeTPURuntime#8133
masahi merged 3 commits into
apache:mainfrom
akmaru:fix-error-edgetpu-runtime

Conversation

@akmaru

@akmaru akmaru commented May 25, 2021

Copy link
Copy Markdown
Contributor

Fixed the errors of EdgeTPURuntime

  • Fixed compile error by the omission include "edgetpu.h"
  • Fixed SEGV in EdgeTPURuntime destructor.
    • It occurs if with destruction by the order EdgeTPUContext -> tflite::Interpreter. It maybe due to implementation of iflite::Interpreter. So, fixed to destruct by the reverse order by using EdgeTPUContext destructor.

@masahi
masahi merged commit 657af3a into apache:main Jun 11, 2021
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Jun 17, 2021
* Fixed the destruction order tflite::Interpreter and EdgeTPUContext

* Fixed include omission

* Formatted
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request Jun 17, 2021
* Fixed the destruction order tflite::Interpreter and EdgeTPUContext

* Fixed include omission

* Formatted
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