RFR: 8367719: Refactor JNI code that uses class_to_verify_considering_redefinition() [v3]

Coleen Phillimore coleenp at openjdk.org
Sat Sep 20 14:36:17 UTC 2025


On Sat, 20 Sep 2025 06:43:00 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> Simplify the boilerplate code in jvm.cpp that calls `JvmtiThreadState::class_to_verify_considering_redefinition()`, and reduce the number of `InstanceKlass::cast()` calls.
>> 
>> I also changed a few fields/arguments from `Klass*` to `InstanceKlass*` as these are used exclusively with `InstanceKlass*`.
>
> Ioi Lam has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:
> 
>  - @sspitsyn comment -- fixed typo
>  - Merge branch 'master' into 8367719-refactor-jni-code-that-uses-class_to_verify_considering_redefinition
>  - @coleenp and @dholmes-ora comments -- simplify implementation; fixed code formatting
>  - Merge branch 'master' into 8367719-refactor-jni-code-that-uses-class_to_verify_considering_redefinition
>  - 8367719: Refactor JNI code that uses class_to_verify_considering_redefinition()

Marked as reviewed by coleenp (Reviewer).

-------------

PR Review: https://git.openjdk.org/jdk/pull/27303#pullrequestreview-3249059326


More information about the serviceability-dev mailing list