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

cpu: aarch64: Enable ACL stateless API for indirect conv #2022

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

theComputeKid
Copy link
Contributor

ACL v24.07 introduces new "stateless" APIs that can be integrated into oneDNN. This PR calls the new API for indirect convolution.

Checklist

General

  • Do all unit and benchdnn tests (make test and make test_benchdnn_*) pass locally for each commit?
  • Have you formatted the code using clang-format?

- Bump ACL requirements to 24.07 and document.
- Call stateless ACL APIs from oneDNN for indirect convolution.
- Update gitignore to handle the .cache folder for clangd code navigation.

Signed-off-by: Hamza Butt <hamza.butt@arm.com>
@theComputeKid theComputeKid requested review from a team as code owners August 2, 2024 13:00
@jondea jondea added the platform:cpu-aarch64 Codeowner: @oneapi-src/onednn-cpu-aarch64 label Aug 2, 2024
@vpirogov vpirogov added this to the v3.6 milestone Aug 2, 2024
Copy link
Contributor

@deb-intel deb-intel left a comment

Choose a reason for hiding this comment

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

README file ok

@theComputeKid
Copy link
Contributor Author

Hi @vpirogov, could we get an accelerated review for this patch? We hope to convert 5-6 operators to our new stateless API by the end of the month, and they all follow the same pattern. An OK from you should satisfy the last codeowner requirement?

@vpirogov vpirogov merged commit 2cfff2b into oneapi-src:main Aug 6, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform:cpu-aarch64 Codeowner: @oneapi-src/onednn-cpu-aarch64
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants