Skip to content
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] Introduce CodegenLoader class #12536

Merged
merged 2 commits into from
Jan 30, 2024

Conversation

jyoungyun
Copy link
Contributor

This commit introduces CodegenLoader class.
This class loads the backend codegen library and provides a function to get the concreate class of ICodegen class.

ONE-DCO-1.0-Signed-off-by: Jiyoung Yun jy910.yun@samsung.com
Co-authored-by: Sanggyu Lee sg5.lee@samsung.com

Related issue : #12505
Draft PR: #12504

This commit introduces CodegenLoader class.
This class loads the backend codegen library and provides a function
to get the concreate class of ICodegen class.

ONE-DCO-1.0-Signed-off-by: Jiyoung Yun <jy910.yun@samsung.com>
Co-authored-by: Sanggyu Lee <sg5.lee@samsung.com>
@jyoungyun jyoungyun added the PR/ready for review It is ready to review. Please review it. label Jan 26, 2024
@jyoungyun jyoungyun requested a review from a team January 26, 2024 06:53
YongseopKim
YongseopKim previously approved these changes Jan 26, 2024
Copy link
Contributor

@YongseopKim YongseopKim left a comment

Choose a reason for hiding this comment

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

LGTM

…ions

ONE-DCO-1.0-Signed-off-by: Jiyoung Yun <jy910.yun@samsung.com>
Copy link
Contributor

@ragmani ragmani left a comment

Choose a reason for hiding this comment

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

LGTM

@glistening glistening merged commit f03abf3 into Samsung:master Jan 30, 2024
9 checks passed
@jyoungyun jyoungyun deleted the onert/introduce_codegen_loader branch January 30, 2024 02:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR/ready for review It is ready to review. Please review it.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants