Skip to content

[CPU][ARM] Call ConvolutionTransformation transformation for "Conv-Add-FQ" subgraph only#36205

Merged
alvoron merged 7 commits into
openvinotoolkit:masterfrom
alvoron:alvoron_conv_transformation_callback_upd3
Jun 3, 2026
Merged

[CPU][ARM] Call ConvolutionTransformation transformation for "Conv-Add-FQ" subgraph only#36205
alvoron merged 7 commits into
openvinotoolkit:masterfrom
alvoron:alvoron_conv_transformation_callback_upd3

Conversation

@alvoron
Copy link
Copy Markdown
Contributor

@alvoron alvoron commented Jun 3, 2026

Details:

  • Call ConvolutionTransformation transformation for Conv-Add-FQ and Conv-FQ subgraphs only
  • Activations between bias and FQ are not supported here yet, some of them will be enabled later

Tickets:

AI Assistance:

  • AI assistance used: no / yes
  • If yes, summarize how AI was used and what human validation was performed (build/tests/manual checks).

@alvoron alvoron requested review from a team as code owners June 3, 2026 08:44
@github-actions github-actions Bot added the category: CPU OpenVINO CPU plugin label Jun 3, 2026
@v-Golubev v-Golubev self-assigned this Jun 3, 2026
Copy link
Copy Markdown
Contributor

@v-Golubev v-Golubev left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread src/plugins/intel_cpu/src/transformations/utils.cpp Outdated
Comment thread src/plugins/intel_cpu/src/transformations/utils.cpp Outdated
Comment on lines +38 to +39
template <typename NodeType>
std::shared_ptr<ov::Node> get_consumer(const ov::Output<NodeType>& output) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
template <typename NodeType>
std::shared_ptr<ov::Node> get_consumer(const ov::Output<NodeType>& output) {
std::shared_ptr<ov::Node> get_consumer(const ov::Output<ov::Node>& output) {

Copy link
Copy Markdown
Contributor Author

@alvoron alvoron Jun 3, 2026

Choose a reason for hiding this comment

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

With this change, CI job is failed:

/__w/openvino/openvino/openvino/src/plugins/intel_cpu/src/transformations/utils.cpp:38:27: note: candidate function not viable: no known conversion from 'Output<const Node>' to 'const Output<ov::Node>' for 1st argument
   38 | std::shared_ptr<ov::Node> get_consumer(const ov::Output<ov::Node>& output) {

https://github.com/openvinotoolkit/openvino/actions/runs/26883907678/job/79291250440?pr=36205#step:6:2691
So, I reverted this change

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

std::shared_ptr<ov::Node> get_consumer(const ov::Output<const ov::Node>& output) {

Can you try this variant?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The last approach works

@alvoron alvoron added the platform: arm OpenVINO on ARM / ARM64 label Jun 3, 2026
@alvoron alvoron added this to the 2026.3 milestone Jun 3, 2026
@alvoron alvoron enabled auto-merge June 3, 2026 14:40
@alvoron alvoron added this pull request to the merge queue Jun 3, 2026
Merged via the queue into openvinotoolkit:master with commit 918b836 Jun 3, 2026
205 of 207 checks passed
@alvoron alvoron deleted the alvoron_conv_transformation_callback_upd3 branch June 3, 2026 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: CPU OpenVINO CPU plugin platform: arm OpenVINO on ARM / ARM64

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants