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

[infra/onert] Disable ACL build as default #14453

Merged
merged 2 commits into from
Dec 16, 2024

Conversation

hseok-oh
Copy link
Contributor

This commit disables ACL build as default.
It will be enabled when target is aarch32/64 and OS is android or linux.

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh [email protected]

This commit disables ACL build as default.
It will be enabled when target is aarch32/64 and OS is android or linux.

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh <[email protected]>
@hseok-oh hseok-oh added the PR/ready for review It is ready to review. Please review it. label Dec 13, 2024
@hseok-oh hseok-oh requested a review from chunseoklee December 13, 2024 06:11
ragmani
ragmani previously approved these changes Dec 13, 2024
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

@@ -1,4 +1,7 @@
# aarch64 android cmake options
#
option(BUILD_ARMCOMPUTE "Build ARM Compute from the downloaded source" ON)
option(DOWNLOAD_ARMCOMPUTE "Build ARM Compute from the downloaded source" ON)
Copy link
Contributor

Choose a reason for hiding this comment

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

It is DOWNLOAD_ARMCOMPUTE, but the description seems to be Build .... It would be better to change to Download..

Suggested change
option(DOWNLOAD_ARMCOMPUTE "Build ARM Compute from the downloaded source" ON)
option(DOWNLOAD_ARMCOMPUTE "Download ARM Compute source" ON)

@hseok-oh
Copy link
Contributor Author

@ys44kim @ragmani @chunseoklee I updated help message. PTAL

Copy link
Contributor

@ys44kim ys44kim left a comment

Choose a reason for hiding this comment

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

LGTM

@hseok-oh hseok-oh merged commit f9f9974 into Samsung:master Dec 16, 2024
9 checks passed
@hseok-oh hseok-oh deleted the default_acl_setting branch December 16, 2024 05:52
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.

4 participants