stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8283997: Unused argument in GraphKit::builtin_throw

Browse Source 
Reviewed-by: roland, redestad, chagedorn
This commit is contained in:
Tobias Hartmann 2022-03-31 10:52:50 +00:00
parent 067b2581d9
commit a11cc97439
3 changed files with 6 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/hotspot/share/opto/graphKit.cpp
View File

@ -526,7 +526,7 @@ void GraphKit::uncommon_trap_if_should_post_on_exceptions(Deoptimization::DeoptR
}
//------------------------------builtin_throw----------------------------------
void GraphKit::builtin_throw(Deoptimization::DeoptReason reason, Node* arg) {
void GraphKit::builtin_throw(Deoptimization::DeoptReason reason) {
bool must_throw = true;
// If this particular condition has not yet happened at this
@ -3334,7 +3334,7 @@ Node* GraphKit::gen_checkcast(Node *obj, Node* superklass,
bool is_aastore = (java_bc() == Bytecodes::_aastore);
Deoptimization::DeoptReason reason = is_aastore ?
Deoptimization::Reason_array_check : Deoptimization::Reason_class_check;
builtin_throw(reason, makecon(TypeKlassPtr::make(objtp->klass())));
builtin_throw(reason);
return top();
} else if (!too_many_traps_or_recompiles(Deoptimization::Reason_null_assert)) {
return null_assert(obj);
@ -3419,7 +3419,7 @@ Node* GraphKit::gen_checkcast(Node *obj, Node* superklass,
bool is_aastore = (java_bc() == Bytecodes::_aastore);
Deoptimization::DeoptReason reason = is_aastore ?
Deoptimization::Reason_array_check : Deoptimization::Reason_class_check;
builtin_throw(reason, load_object_klass(not_null_obj));
builtin_throw(reason);
}
} else {
(*failure_control) = not_subtype_ctrl;

8
src/hotspot/share/opto/graphKit.hpp
View File

@ -266,12 +266,8 @@ class GraphKit : public Phase {
JVMState* transfer_exceptions_into_jvms();
// Helper to throw a built-in exception.
// Range checks take the offending index.
// Cast and array store checks take the offending class.
// Others do not take the optional argument.
// The JVMS must allow the bytecode to be re-executed
// via an uncommon trap.
void builtin_throw(Deoptimization::DeoptReason reason, Node* arg = NULL);
// The JVMS must allow the bytecode to be re-executed via an uncommon trap.
void builtin_throw(Deoptimization::DeoptReason reason);
// Helper to check the JavaThread::_should_post_on_exceptions flag
// and branch to an uncommon_trap if it is true (with the specified reason and must_throw)

2
src/hotspot/share/opto/parse2.cpp
View File

@ -194,7 +194,7 @@ Node* Parse::array_addressing(BasicType type, int vals, const Type*& elemtype) {
// If we have already recompiled with the range-check-widening
// heroic optimization turned off, then we must really be throwing
// range check exceptions.
builtin_throw(Deoptimization::Reason_range_check, idx);
builtin_throw(Deoptimization::Reason_range_check);
}
}
}