Skip to content

Align AArch64 ACL integration with oneDNN v3.10.x#313

Open
allnes wants to merge 1 commit into
openvinotoolkit:v3.10_for_ie_masterfrom
allnes:anesterov/align-arm-uxl-dnn
Open

Align AArch64 ACL integration with oneDNN v3.10.x#313
allnes wants to merge 1 commit into
openvinotoolkit:v3.10_for_ie_masterfrom
allnes:anesterov/align-arm-uxl-dnn

Conversation

@allnes
Copy link
Copy Markdown
Collaborator

@allnes allnes commented Jun 2, 2026

Summary

This PR aligns the OpenVINO oneDNN fork delta for AArch64 ACL support with the v3.10.x integration branch.

The change makes the ACL integration explicitly AArch64-only, moves the ACL-backed CPU primitive code into the AArch64 CPU implementation area, and updates the build and primitive registration paths to use the AArch64-specific ACL configuration.

Technical details

  • Replaces the generic DNNL_USE_ACL build option with DNNL_AARCH64_USE_ACL.
  • Removes the 32-bit ARM target path from the forked oneDNN CPU architecture selection.
  • Moves ACL-backed primitive sources from src/cpu/acl into src/cpu/aarch64 and updates namespaces, includes, and CMake source selection accordingly.
  • Registers ACL implementations through AArch64-specific guards, including the ACL reorder implementation used by OpenVINO CPU plugin layout conversions.
  • Updates AArch64 CPU primitive lists, ACL threading integration, platform flag handling, and JIT profiling defaults to match the AArch64-only ACL model.
  • Updates AArch64 build and CI workflow definitions to use the new option name and target architecture contract.

Scope

The PR is intended to provide the oneDNN-side dependency revision consumed by OpenVINO through src/plugins/intel_cpu/thirdparty/onednn. It does not require OpenVINO to use an external oneDNN source tree.

openvinotoolkit/openvino#36193

Drop 32-bit ARM target support and the old DNNL_USE_ACL option, move ACL sources back under the AArch64 layout, and keep only the fork-only AArch64 ACL behavior needed by the OpenVINO fork.

Update AArch64 ACL CI automation to use the v3.10.x-compatible DNNL_AARCH64_USE_ACL option.
@allnes allnes changed the title Align AArch64 ACL fork delta with oneDNN v3.10.x Align AArch64 ACL integration with oneDNN v3.10.x Jun 2, 2026
@dzarukin
Copy link
Copy Markdown

dzarukin commented Jun 2, 2026

Unrelated to PR question: wouldn't it be cheaper to migrate to v3.12 instead of cherry-picking all those necessary changes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants