On Tue, 24 Feb 2026 00:32:48 GMT, Leonid Mesnik <[email protected]> wrote:
>> Hi >> >> Please review following fix that remove incorrect assertion. >> >> Verification should be always done for redefined classes, and is called >> `bool Verifier::verify(InstanceKlass* klass, bool should_verify_class, >> TRAPS)` >> where `should_verify_class` and `is_eligible_for_verification` define if >> verification should be done or not. >> Thus, seems it doesn't make a sense to check `BytecodeVerificationRemote` >> during verification. >> >> Verified that class redefinition tests pass now with >> TEST_VM_OPTS="-XX:-BytecodeVerificationRemote" and fastdebug > > Leonid Mesnik 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 three additional > commits since the last revision: > > - Updated test to verify that JVMTI works with > -XX:-BytecodeVerificationRemote > - Merg branch 'master' of https://github.com/openjdk/jdk into 8376295 > - 8376295: "assert(BytecodeVerificationRemote) failed: Should not be here" > when running class redefinition test with -XX:-BytecodeVerificationRemote Thanks for adding the test. Looks good. ------------- Marked as reviewed by coleenp (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/29476#pullrequestreview-3847779539
