-
Notifications
You must be signed in to change notification settings - Fork 159
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[onert] Relocate API functions #14416
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
__API_NNFW_API_INTERNAL_H__
-> __API_NNFW_SESSION_H__
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This commit relocates API functions - Rename internal API implementation file from nnfw_debug.cc to nnfw_internal.cc - Move some internal API implementation to src/nnfw_internal.cc - Rename nnfw_api_internal.h/cc to nnfw_session.h/cc ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh <[email protected]>
2691bae
f736dff
to
2691bae
Compare
@chunseoklee @glistening @zetwhite I rebased to resolve conflict. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This commit relocates API functions
ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh [email protected]