[CPU][ARM] Align CPU plugin with AArch64 oneDNN ACL integration#36193
Open
allnes wants to merge 6 commits into
Open
[CPU][ARM] Align CPU plugin with AArch64 oneDNN ACL integration#36193allnes wants to merge 6 commits into
allnes wants to merge 6 commits into
Conversation
Use a local migrated oneDNN source tree via OV_CPU_ONEDNN_SOURCE_DIR, switch CPU plugin integration to DNNL_AARCH64_USE_ACL, and remove obsolete ARM32/ACL32 CPU plugin paths. Keep the OpenVINO changes scoped to CPU plugin, oneDNN integration, ARM validation, and the ARM64 CPU functional skip needed for ACL f16 ReduceProd accuracy.
# Conflicts: # src/plugins/intel_cpu/src/transformations/transformation_pipeline.cpp
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR aligns the OpenVINO CPU plugin with the updated AArch64 ACL integration in oneDNN and updates the bundled oneDNN submodule to the corresponding dependency revision.
The oneDNN update changes the ACL integration model to be AArch64-specific. The OpenVINO CPU plugin is updated to consume that model through the existing thirdparty submodule and to remove obsolete 32-bit ARM ACL integration paths that no longer match the oneDNN v3.10.x fork layout.
Technical details
src/plugins/intel_cpu/thirdparty/onednnto the oneDNN AArch64 ACL integration revision.DNNL_USE_ACLtoDNNL_AARCH64_USE_ACL.DNNL_TARGET_ARCH=AARCH64for ARM64 builds and removes the stale 32-bit ARM oneDNN target path from the CPU plugin integration.src/plugins/intel_cpu/thirdparty/onednn; no external oneDNN source tree override is introduced or required.Scope
The OpenVINO-side change is limited to the CPU plugin integration required by the updated oneDNN dependency and to the associated ARM/AArch64 test and validation configuration. Build artifacts, local CMake cache state, and external oneDNN source paths are not part of the change.
openvinotoolkit/oneDNN#313