On Tue, 13 Sep 2022 13:29:45 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:
> I moved misc_flags out to their own misc flags class so that we can put the > writeable accessFlags there too with atomic access. > > Tested with tier1-4. Nice refactoring and simplification. It looks good to me. Thanks, Serguei src/hotspot/share/prims/jvmtiRedefineClasses.cpp line 4408: > 4406: if (!the_class->has_been_redefined()) { > 4407: the_class->set_has_been_redefined(); > 4408: } Nit: Is this change really needed? ------------- Marked as reviewed by sspitsyn (Reviewer). PR: https://git.openjdk.org/jdk/pull/10249