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][ARM] JIT Floor Mod Operation #27706

Merged
merged 8 commits into from
Nov 29, 2024

Conversation

geeky33
Copy link
Contributor

@geeky33 geeky33 commented Nov 22, 2024

Details:

  • Added JIT emitter for Eltwise Floor Mod operation on ARM64 SIMD
  • Implemented fp32 optimization replacing C++ Math implementation
  • Modified ARM64 executor to support new JIT emitter
  • Updated kernel files to include Floor Mod in Eltwise operations
  • Added test coverage for JIT implementation verification
  • Transitioned operation type from Math to Eltwise for better performance

@a-sidorova can you please review the code ? :)

Tickets:

image

@geeky33 geeky33 requested review from a team as code owners November 22, 2024 16:07
@sys-openvino-ci sys-openvino-ci added the ExternalPR External contributor label Nov 22, 2024
@github-actions github-actions bot added the category: CPU OpenVINO CPU plugin label Nov 22, 2024
@a-sidorova a-sidorova self-assigned this Nov 25, 2024
Copy link
Contributor

@a-sidorova a-sidorova left a comment

Choose a reason for hiding this comment

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

@geeky33 Thank you for the contribution to OpenVINO! 👍🏼

Also, could you please launch CPU-specific tests using --gtest_filter='*EltwiseLayerCPUTest*FloorMod*'? These tests verify that JIT primitive with your emitter is used now.
I believe that there will be 0 such tests. Then we need to add them 😊

  • Please, update the variable eltwiseOpTypesBinInp() in tests - Floor_mod should be tested on x64/x86 platforms as before and on ARM64 machines too. If you do it and launch tests with the mentioned gtest filter above, the tests should be failed with error message unexpected primitive type.
  • Please update the method getPrimitiveType - now it should return jit for Floor_mod. After that, tests should be green 😃

@geeky33
Copy link
Contributor Author

geeky33 commented Nov 28, 2024

Hi @a-sidorova i have made a commit as you told me .
Could you please review the code?

@a-sidorova
Copy link
Contributor

build_jenkins

@a-sidorova
Copy link
Contributor

build_jenkins

@a-sidorova
Copy link
Contributor

build_jenkins

Copy link
Contributor

@a-sidorova a-sidorova left a comment

Choose a reason for hiding this comment

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

Thank you for the effort!

@dmitry-gorokhov dmitry-gorokhov added this pull request to the merge queue Nov 29, 2024
Merged via the queue into openvinotoolkit:master with commit 11db1e1 Nov 29, 2024
170 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: CPU OpenVINO CPU plugin ExternalPR External contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Good First Issue] [ARM]: Implement CPU plugin just-in-time emitter for FloorMod operation
4 participants