Skip to content

[Snippets][CPU] Enable AArch64 clang-tidy validation#31203

Merged
EgorDuplensky merged 4 commits intoopenvinotoolkit:masterfrom
aobolensk:tidy-aarch64
Jul 8, 2025
Merged

[Snippets][CPU] Enable AArch64 clang-tidy validation#31203
EgorDuplensky merged 4 commits intoopenvinotoolkit:masterfrom
aobolensk:tidy-aarch64

Conversation

@aobolensk
Copy link
Copy Markdown
Contributor

@aobolensk aobolensk commented Jul 3, 2025 •
edited
Loading

Details:

Enable clang-tidy run on aarch64 platform for CPU plugin + snippets

Tickets:

  • N/A

@aobolensk aobolensk added the platform: arm OpenVINO on ARM / ARM64 label Jul 3, 2025
@github-actions github-actions bot added category: CPU OpenVINO CPU plugin category: CI OpenVINO public CI github_actions Pull requests that update GitHub Actions code labels Jul 3, 2025
@aobolensk aobolensk force-pushed the tidy-aarch64 branch 6 times, most recently from b23d92a to 7e8dd69 Compare July 4, 2025 08:11
@aobolensk aobolensk marked this pull request as ready for review July 4, 2025 12:09
@aobolensk aobolensk requested review from a team as code owners July 4, 2025 12:09
@aobolensk aobolensk changed the title [CPU] Enable AArch64 clang-tidy validation [Snippets][CPU] Enable AArch64 clang-tidy validation Jul 4, 2025
EgorDuplensky
EgorDuplensky reviewed Jul 8, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@EgorDuplensky EgorDuplensky left a comment

Choose a reason for hiding this comment

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

Please double check the commented function arguments which should be replaced with maybe_unused

@aobolensk
Copy link
Copy Markdown
Contributor Author

aobolensk commented Jul 8, 2025

Please double check the commented function arguments which should be replaced with maybe_unused

Double checked, thanks!

@EgorDuplensky EgorDuplensky enabled auto-merge July 8, 2025 17:02
@EgorDuplensky EgorDuplensky added this pull request to the merge queue Jul 8, 2025
Merged via the queue into openvinotoolkit:master with commit a4744b1 Jul 8, 2025
210 checks passed
@alvoron alvoron mentioned this pull request Apr 1, 2026
Merged
github-merge-queue bot pushed a commit that referenced this pull request Apr 2, 2026
@alvoron
[CPU][ARM] AclPoolingExecutor should not use default parameter valu…
35d22f2 
…es if provided ones are incorrect (#35110)

### Details:
- Partially revert changes introduced in
#31203 to keep parameter
validation passed into executors more strict

### Tickets:
 - *ticket-id*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@EgorDuplensky EgorDuplensky EgorDuplensky approved these changes

Assignees

@EgorDuplensky EgorDuplensky

@v-Golubev v-Golubev

Labels

category: CI OpenVINO public CI category: CPU OpenVINO CPU plugin github_actions Pull requests that update GitHub Actions code platform: arm OpenVINO on ARM / ARM64

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@aobolensk @EgorDuplensky @v-Golubev