Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
8211170: AArch64: Warnings in C1 and template interpreter
Backport-of: 08828a5
  • Loading branch information
Vladimir Kempik committed Dec 15, 2021
1 parent c94151b commit 9363d55
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 1 deletion.
1 change: 1 addition & 0 deletions src/hotspot/cpu/aarch64/c1_LIRAssembler_aarch64.cpp
Expand Up @@ -1732,6 +1732,7 @@ void LIR_Assembler::arith_op(LIR_Code code, LIR_Opr left, LIR_Opr right, LIR_Opr
default: ShouldNotReachHere();
}
break;
default:
ShouldNotReachHere();
}
} else {
Expand Down
6 changes: 5 additions & 1 deletion src/hotspot/cpu/aarch64/c1_LIRGenerator_aarch64.cpp
Expand Up @@ -584,8 +584,8 @@ void LIRGenerator::do_ArithmeticOp(ArithmeticOp* x) {
case doubleTag: do_ArithmeticOp_FPU(x); return;
case longTag: do_ArithmeticOp_Long(x); return;
case intTag: do_ArithmeticOp_Int(x); return;
default: ShouldNotReachHere(); return;
}
ShouldNotReachHere();
}

// _ishl, _lshl, _ishr, _lshr, _iushr, _lushr
Expand Down Expand Up @@ -792,9 +792,13 @@ void LIRGenerator::do_MathIntrinsic(Intrinsic* x) {
__ abs(value.result(), dst, LIR_OprFact::illegalOpr);
break;
}
default:
ShouldNotReachHere();
}
break;
}
default:
ShouldNotReachHere();
}
}

Expand Down
2 changes: 2 additions & 0 deletions src/hotspot/cpu/aarch64/templateTable_aarch64.cpp
Expand Up @@ -2323,6 +2323,7 @@ void TemplateTable::resolve_cache_and_index(int byte_no,
switch (code) {
case Bytecodes::_nofast_getfield: code = Bytecodes::_getfield; break;
case Bytecodes::_nofast_putfield: code = Bytecodes::_putfield; break;
default: break;
}

assert(byte_no == f1_byte || byte_no == f2_byte, "byte_no out of range");
Expand Down Expand Up @@ -2947,6 +2948,7 @@ void TemplateTable::jvmti_post_fast_field_mod()
case Bytecodes::_fast_dputfield: __ pop_d(); break;
case Bytecodes::_fast_fputfield: __ pop_f(); break;
case Bytecodes::_fast_lputfield: __ pop_l(r0); break;
default: break;
}
__ bind(L2);
}
Expand Down

1 comment on commit 9363d55

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

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

Please sign in to comment.