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] [ARM64] jit emitter pipeline fix #23387

Conversation

eshoguli
Copy link
Contributor

Details:

  • [CPU] [ARM64] jit emitter pipeline fix

@eshoguli eshoguli requested review from a team as code owners March 11, 2024 23:59
@github-actions github-actions bot added category: IE Tests OpenVINO Test: plugins and common category: CPU OpenVINO CPU plugin labels Mar 11, 2024
@eshoguli eshoguli force-pushed the es/cpu/arm_jit_emitters_pipeline_fix branch 2 times, most recently from e2b2158 to c690e53 Compare March 12, 2024 00:05
@github-actions github-actions bot removed the category: IE Tests OpenVINO Test: plugins and common label Mar 12, 2024
@eshoguli eshoguli force-pushed the es/cpu/arm_jit_emitters_pipeline_fix branch 5 times, most recently from 5115e00 to f5eba84 Compare March 12, 2024 17:43
push_arg_entry_of("exp_pol5", 0x3c07cfce, true);
}

std::set<std::vector<element::Type>> jit_exp_emitter::get_supported_precisions(const std::shared_ptr<ov::Node>& node) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Just detail

Suggested change
std::set<std::vector<element::Type>> jit_exp_emitter::get_supported_precisions(const std::shared_ptr<ov::Node>& node) {
std::set<element::TypeVector> jit_exp_emitter::get_supported_precisions(const std::shared_ptr<ov::Node>& node) {

@eshoguli eshoguli force-pushed the es/cpu/arm_jit_emitters_pipeline_fix branch from f5eba84 to 6b42b2d Compare March 13, 2024 11:38
@ilya-lavrenov ilya-lavrenov added the platform: arm OpenVINO on ARM / ARM64 label Mar 13, 2024
@eshoguli eshoguli force-pushed the es/cpu/arm_jit_emitters_pipeline_fix branch from 6b42b2d to be2b24a Compare March 14, 2024 10:16
@dmitry-gorokhov dmitry-gorokhov added this to the 2024.1 milestone Mar 14, 2024
@dmitry-gorokhov dmitry-gorokhov self-assigned this Mar 14, 2024
@eshoguli eshoguli force-pushed the es/cpu/arm_jit_emitters_pipeline_fix branch 2 times, most recently from f5eba84 to af36ad5 Compare March 21, 2024 11:30
@eshoguli eshoguli mentioned this pull request Mar 21, 2024
@eshoguli eshoguli force-pushed the es/cpu/arm_jit_emitters_pipeline_fix branch 2 times, most recently from 6f31719 to 7fae5f9 Compare March 23, 2024 00:52
@dmitry-gorokhov
Copy link
Contributor

@xuchen-intel Please review the changes

@eshoguli eshoguli force-pushed the es/cpu/arm_jit_emitters_pipeline_fix branch from 11b12b6 to 2182be7 Compare March 25, 2024 11:30
@eshoguli eshoguli force-pushed the es/cpu/arm_jit_emitters_pipeline_fix branch 2 times, most recently from 0c990ea to 008ad9a Compare March 25, 2024 17:01
@eshoguli eshoguli requested a review from xuchen-intel March 25, 2024 17:06
@eshoguli eshoguli force-pushed the es/cpu/arm_jit_emitters_pipeline_fix branch from 008ad9a to 94ec445 Compare March 25, 2024 17:54
size_t _idx = end_gpr_idx - gpr_idx; // we allocate from the end

if (aux_gpr_idxs.size() >= get_aux_gprs_count()) break;
if ((_idx == Xbyak_aarch64::Operand::X18) ||
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you please clarify why these registers cannot bu used? Worth to add comment with the description. CAn be done in separate PR

@dmitry-gorokhov dmitry-gorokhov added this pull request to the merge queue Mar 28, 2024
Merged via the queue into openvinotoolkit:master with commit f0ebba0 Mar 28, 2024
103 checks passed
@dmitry-gorokhov dmitry-gorokhov deleted the es/cpu/arm_jit_emitters_pipeline_fix branch March 28, 2024 06:56
bbielawx pushed a commit to bbielawx/openvino that referenced this pull request Apr 12, 2024
### Details:
 - *[CPU] [ARM64] jit emitter pipeline fix*
alvoron pushed a commit to alvoron/openvino that referenced this pull request Apr 29, 2024
### Details:
 - *[CPU] [ARM64] jit emitter pipeline fix*
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.

5 participants