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

Make dotprod and i8mm configurable features #1335

Merged
merged 1 commit into from
Aug 21, 2024
Merged

Make dotprod and i8mm configurable features #1335

merged 1 commit into from
Aug 21, 2024

Conversation

rinon
Copy link
Collaborator

@rinon rinon commented Aug 13, 2024

Adds the asm_arm64_dotprod and asm_arm64_i8mm features. We previously assumed that dotprod and i8mm were always enabled when building for aarch64 and then disabled at runtime if not supported. We also want to support the configuration where we do not even build the code, so these added features (enabled by default) allow this to be configurable.

@rinon rinon requested a review from randomPoison August 13, 2024 20:50
Adds the `asm_arm64_dotprod` and `asm_arm64_i8mm` features. We previously assumed that dotprod and i8mm were always enabled when building for aarch64 and then disabled at runtime if not supported. We also want to support the configuration where we do not even build the code, so these added features (enabled by default) allow this to be configurable.
@rinon rinon force-pushed the sjc/arm64_features branch from d1191a3 to 07c27cb Compare August 13, 2024 20:51
Copy link
Collaborator

@kkysen kkysen left a comment

Choose a reason for hiding this comment

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

What environments is this for?

@rinon rinon merged commit 110ebd4 into main Aug 21, 2024
28 checks passed
@rinon
Copy link
Collaborator Author

rinon commented Aug 21, 2024

Android does not build with dotprod or i8mm support.

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.

2 participants